Conversation
The `basecamp files uploads create` command was missing from both the Quick Reference table and the Files & Documents resource section, causing AI agents to guess incorrect command names (e.g. `basecamp files upload`) when asked to upload files. This led to silent failures that were hard to diagnose.
There was a problem hiding this comment.
Pull request overview
Tip
If you aren't ready for review, convert to a draft PR.
Click "Convert to draft" or run gh pr ready --undo.
Click "Ready for review" or run gh pr ready to reengage.
Updates the Basecamp skill documentation to include the correct CLI path for uploading files, preventing agents from using non-existent/incorrect upload commands.
Changes:
- Adds
basecamp files uploads createto the Quick Reference table. - Adds root and folder-targeted upload examples to the “Files & Documents” command list.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
There was a problem hiding this comment.
1 issue found across 1 file
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="skills/basecamp/SKILL.md">
<violation number="1" location="skills/basecamp/SKILL.md:162">
P2: Missing `--json` flag — every other create/write command in the Quick Reference table includes it. Without `--json`, agents won't receive structured output from uploads, which defeats the PR's goal of guiding agents to the correct command usage.</violation>
</file>
Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.
There was a problem hiding this comment.
Pull request overview
Adds the missing Basecamp CLI upload command to the Basecamp skill documentation so agents use the correct invocation and can optionally target a folder via --vault.
Changes:
- Add
basecamp files uploads createto the Quick Reference table, showing--vaultas optional. - Add upload examples (root and folder-targeted) to the “Files & Documents” command section.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
…evanmiller/feature-gap-analysis * origin/main: Show full content in detail views instead of truncating at 40 chars (basecamp#338) Switch CODEOWNERS from sip to cli team (basecamp#346) Fix dock ordering and add --all flag to `projects show` (basecamp#333) Add file upload command to SKILL.md (basecamp#343) Replace "pending" with "incomplete" in todos output; omit from default view (basecamp#327) Add `--in` as global alias for `--project` (basecamp#334) Fix 3 post-QA issues on api command (basecamp#330) deps: bump the go-dependencies group with 2 updates (basecamp#331) Improve tool instance disambiguation format (basecamp#329) Fix emoji/CJK alignment in search results (basecamp#328)
Summary
basecamp files uploads createcommand was missing from both the Quick Reference table and the Files & Documents resource section inSKILL.mdbasecamp files upload, which silently returnednullwithout uploading anything--vaultfor folder targeting) to both locations so agents use the correct command pathBefore this update Claude failed to upload any files. Afterwards it had no problems.
Summary by cubic
Add the missing
basecamp files uploads createcommand toSKILL.md(Quick Reference and Files & Documents), and update the Quick Reference to include--jsonand optional--vault([--vault <folder_id>]). This prevents incorrect usage (e.g.,basecamp files upload) that previously did nothing.Written for commit 9c06231. Summary will update on new commits.