docs(archdev-skill): tag every archdev log post with its project - #30
Merged
Merged
Conversation
`archdev log post` now takes --project <id> (firstlanding PR #15014) and `archdev projects list --query` finds the project (PR #14977). Tell agents to look the project up before the first lifecycle post, pass its ID on every post, and never create a project per PR, task, or session. Every `log post` example in SKILL.md section 3 and monitor.md carries the flag. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
ark-archastro
marked this pull request as ready for review
September 24, 2026 01:46
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: 3fdfb5898d
ℹ️ 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".
… a steer Review feedback on the project-tag text. Bootstrap now requires 0.46.0, the release that ships `projects` and `log post --project`, and probes both before accepting an installed binary, so existing 0.45.x installs are upgraded instead of posting untagged forever. The lookup is per piece of work, not per session: a steer to a different initiative means looking the project up again before the next post. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
monitor.md already said it; the SKILL.md beat-2 summary, which agents read first, only said to look one up. 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
Slice 2 of the ArchDev projects plan added
archdev log post --project <id>(firstlanding #15014), on top ofarchdev projects list|show|create|updatefrom slice 1 (#14977). The CLI warns when a post carries no project but does not refuse, so the skill text is what actually gets agents tagging their posts. This is the skill half of that change.Requires CLI 0.46.0, the release that ships both
projectsandlog post --project(published 2026-09-24 from firstlanding 25faed7884, which includes both PRs). The bootstrap scripts now enforce that floor and upgrade older installs, so the skill text and the CLI cannot drift apart on a machine that runs bootstrap.What changed
archdev/SKILL.mdsection 3, beat 2: before the first post, find the project witharchdev projects list --query "<subject>"and pass its ID as--project <id>on everyarchdev log postin the session.archdev/references/monitor.md:--kindposts alike; create a project only when no active one fits, named for the product area or initiative; never one per PR, task, or session. Explains that the tag lands inmetadata.project_idbeside thepull_request/task_idjoin keys, that there is no default, and that an untagged post warns and shows under Unfiled.archdev log postexample (lifecycle table examples, reply, merged--kind done --event pr.closed, free-text, and the structured Report step 5) carries--project.prj_shape only and never looks the project up on a post.archdev/scripts/bootstrap.sh,bootstrap.ps1,references/bootstrap.md: minimum version 0.46.0, andsupports_skill/Test-Skillprobeprojects list --helpandlog post --helpfor--project <id>before accepting an installed binary.Rebased on
mainafter cc6592e (review-annotations text); the SKILL.md beat-2 paragraph keeps both additions.Risk
Low. Docs plus the bootstrap version floor. Raising the floor means every machine that runs bootstrap upgrades to 0.46.0 on its next session start; that is the intended effect.
Testing
Read-through plus probe checks. The examples match
archdev log post --helpon a CLI built from firstlanding main at cf41c32751, each line parses as shell (one added token per line, quoting unchanged),bash -npasses on bootstrap.sh, and the two new probe greps were run against that build (match) and against an installed 0.45.9 (no match). bootstrap.ps1 was not executed (no PowerShell here).🤖 Generated with Claude Code