Repository navigation
shots skill: pass --project to shots upload for the Company Room post - #66
Merged
Merged
Conversation
…mpany Room post archdev 0.53.0 makes `archdev shots upload` post the run's stills to the organization's Company Room after the Media upload. Section 6 of the skill still showed the bare command, so agents following it posted untagged and got the CLI's warning. The section now puts `--project <prj_id>` in the example, explains the post and its `stream_post.status` values (posted, unchanged, retagged, no_room), says how to retag an untagged post by rerunning with `--project`, and tells agents not to also attach the PNGs to a `done` post. Plugin mirrors regenerated with `npm run build`. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 2ea8cd6f0a
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
The upload example now uses `--project`, which 0.52.x rejects as an unknown argument, so the header's minimum moves to 0.53.0 (released 2026-10-13) and tells agents to check `archdev --version` and ask the person to run `archdev upgrade`. The subsection no longer carries its own version note. Plugin mirrors regenerated. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Review on ArchCode
Problem and author intent
archdev CLI v0.53.0 (released 2026-10-13, firstlanding #18144) makes
archdev shots uploadpost the run's stills to the organization's Company Room after the Media upload, filed under--project <prj_id>like every other stream post. Section 6 of the shots skill still shows the bareshots uploadcommand and does not mention the post, so an agent that follows it:archdev shots upload --feature <f> --root <root>with no--project;stream <status>means.Teammates get screenshots in the stream, but outside any project, and agents may also attach the same PNGs to a
donepost with-a, duplicating them.What changed
shots/SKILL.mdsection 6, mirrored into everyplugins/*/skills/shots/SKILL.mdbynpm run build:--project <prj_id>, and the text says the upload posts the stills to the Company Room and how to find the project ID (same ID asarchdev log post).stream_post.statusvaluesposted,unchanged,retaggedandno_room, how to retag an untagged post by rerunning with--project, and that a failed post is retried onto one message by rerunning.doneorstartpost.stream <status> [<message-id>],--jsonaddsstream_post.The header's minimum moves from 0.52.1 to 0.53.0, since the example now uses a flag 0.52.x rejects; it tells agents to check
archdev --versionand ask the person to runarchdev upgrade. The catalogversion(0.49.7) is not bumped; this is a skill text change, not a CLI release.Scope: docs only (skill text and generated plugin mirrors).
Risk: low. Prose that agents read; no code or hook wiring changes. The regenerated mirrors are byte-identical copies made by the checked generator.
User impact: agents following the skill now file their screenshot posts under the right project and can fix an untagged post.
Testing
tools/plugin-gen:npm run build,npm run check(132 generated files match),npm run typecheck,npm test(62 pass). TheHarness pluginsworkflow runs the same on this PR.archdev shots upload --helplists--project <id>) and by the firstlanding CLI e2etests/integration/shots_upload_e2e.rs(posted, unchanged, retagged, no_room, failed-post retry), which shipped with #18144.🤖 Generated with Claude Code