Skip to content

Port AGENTS.md to APM local path dependency#48

Merged
srid merged 10 commits intomainfrom
port-agents-to-apm
Apr 6, 2026
Merged

Port AGENTS.md to APM local path dependency#48
srid merged 10 commits intomainfrom
port-agents-to-apm

Conversation

@srid
Copy link
Copy Markdown
Member

@srid srid commented Apr 6, 2026

Replaces AGENTS.md with APM's local path dependency feature (microsoft/apm#270) so project instructions deploy to .claude/rules/ alongside dependency instructions — no apm compile workaround needed.

A local/ directory acts as an APM package: apm.yml at root, instructions in .apm/instructions/. The project's apm.yml declares it as ./local, and apm install -t claude deploys everything into .claude/rules/ in one shot. A new just apm target wraps this.

The git policy rule ("no rebase, no force push") was intentionally dropped — it already lives in the global ~/.claude/CLAUDE.md.

srid added 3 commits April 6, 2026 15:24
Move project-specific build/CI instructions from AGENTS.md into
.apm/instructions/ (APM source of truth) and a root CLAUDE.md
(for Claude Code). The git policy rule is omitted since it's
already in the global ~/.claude/CLAUDE.md.
- Add `applyTo: "**"` to project instruction (required for apm compile)
- Add `target: claude` to apm.yml
- Add `just apm` target: runs `apm install` + `apm compile`
- Gitignore generated CLAUDE.md (alongside .claude/)
- Remove manually created CLAUDE.md (now generated by apm compile)
@srid srid changed the title Port AGENTS.md instructions to APM Port AGENTS.md to APM with justfile target Apr 6, 2026
@srid
Copy link
Copy Markdown
Member Author

srid commented Apr 6, 2026

Execute Results

Step Status Duration Verification
sync 6s git fetch ran without error
research 62s Understood APM instructions format from agency package
hickey 20s Dropped redundant git policy (already in global CLAUDE.md)
branch 39s Draft PR #48 created on port-agents-to-apm
implement 10m 11s Created .apm/instructions/, justfile target, gitignore updates
docs 6s No documentation files documented; skipped
police 8s All changes are config/docs; skipped
fmt 0s No format command documented; skipped
commit 20s Commits a24211a + 774917c pushed
test 0s Config-only changes; skipped
ci 9m 37s vira ci -b passed; GitHub checks passed
update-pr 5s PR title/body updated to match final scope
Total ~20m

Optimization suggestions

  • implement was the bottleneck (10m) — spent time discovering apm compile is the correct path for local instructions and that applyTo is required. Pre-reading APM docs before invoking /do would cut this significantly.
  • ci took 9m+ — inherent to vira ci -b building 3 flakes. No shortcut here.
  • research + implement overlap — the APM tooling exploration could have been done in research rather than during implement.

Workflow completed at 2026-04-06T19:42:18Z.

Leverage APM's local filesystem path dependency support (PR #270)
to deploy project instructions to .claude/rules/ via `apm install`
directly — no `apm compile` workaround needed.

Structure: local/ is an APM package with apm.yml + .apm/instructions/,
declared as `./local` in the project's apm.yml dependencies.
@srid srid changed the title Port AGENTS.md to APM with justfile target Port AGENTS.md to APM local path dependency Apr 6, 2026
srid added 3 commits April 6, 2026 16:02
just apm now reads AI_AGENT env var (default: claude) to pick
the apm install target. just agent runs apm then launches the
coding agent.
Rename local/ to agent/, add agent/mod.just with apm and run
targets. Root justfile imports via `mod agent` — usage becomes
`just agent::apm` and `just agent::run`.
@srid
Copy link
Copy Markdown
Member Author

srid commented Apr 6, 2026

microsoft/apm#595

@srid srid marked this pull request as ready for review April 6, 2026 23:01
@srid srid merged commit b221764 into main Apr 6, 2026
4 checks passed
@srid srid deleted the port-agents-to-apm branch April 6, 2026 23:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant