feat: complete AGENT, INSTALL modules; promote ORCH to Ready#33
Conversation
authored-by: aneki (@joshuaboys)
Bring over architecture decisions, designs, module specs, research, and reviews that represent current APS direction. Skipped 24 stale files (completed specs, execution artifacts, templates already here). Authored-By: Aneki (joshuaboys)
Add per-tool usage examples to agent docs (Claude Code, Copilot, OpenCode, Codex, Gemini). Update onboarding design to reflect architecture change from OpenTUI (Bun/Zig) to Rust/Ratatui. Mark AGENT-005 complete. APS: AGENT-005 Authored-By: Aneki (joshuaboys)
Update OpenCode model IDs from claude-opus-4-20250514 to claude-opus-4-6 and claude-sonnet-4-20250514 to claude-sonnet-4-6. Add vendor comments to Codex o4-mini config. Create cross-harness test plan documenting format validation and manual test procedures for all 5 tool harnesses. Mark AGENT module as Complete. authored-by: aneki (@joshuaboys)
…tion Build the complete INSTALL module: - aps init: 3-step shell-prompt wizard (profile, scope, AI tools) producing .aps/ layout with config.yml. Non-interactive fallback via flags. - aps migrate: v1→v2 conversion with dry-run, file moves, backup to .aps/backup/, hook path rewriting, and config inference. - aps update: v2-aware, reads config.yml to refresh tool-specific files. - Multi-tool support for all 6 targets (Claude Code, Copilot, Codex, OpenCode, Gemini, generic) with correct file placement. - New aps-rules-v2.md template (plans/designs/, project-context.md). - New project-context.md template with TODO markers for agent bootstrap. - APS_LOCAL env var for local development testing. Mark all 9 INSTALL work items complete. Add decisions D-022 through D-025 to the index. authored-by: aneki (@joshuaboys)
Resolve all 5 open decisions: coexist with TASKS (D-001), inline learnings (D-002), advisory VCS only (D-003), TypeScript MCP server (D-004), ephemeral context packages (D-005). Add confidence and detailed validation to all work items. Promote from Draft to Ready. authored-by: aneki (@joshuaboys)
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: d1a519bd64
ℹ️ 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".
- Install codex-config-snippet.toml during Codex scaffold so users can complete the documented merge step - Use v2_install_cli/v2_install_scripts during migration instead of copying stale v1 binaries - Remove only known APS files from lib/ during migration, preserving any non-APS project files that may coexist in the directory authored-by: aneki (@joshuaboys)
Review Feedback AddressedCode Changes (1 commit pushed)
All 3 threads resolved. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 09e0c1cb5d
ℹ️ 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".
authored-by: aneki (@joshuaboys)
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 8e312b9aac
ℹ️ 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".
- Redirect prompt_select/prompt_multi display output to stderr so command substitution captures only the selection value - Exit 1 (not 0) when cmd_migrate finds no v1 installation - Strengthen is_v1_layout to require ≥2 APS markers, preventing false positives on repos with unrelated bin/aps - Use cp -a designs/. to include dotfiles during migration Authored-By: Aneki (joshuaboys)
Review Feedback AddressedCode Changes (1 commit pushed)
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 2a97a6da14
ℹ️ 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".
- Back up existing aps-rules.md to .aps/backup/ before overwriting with v2 template - Back up aps-planning/scripts/ to .aps/backup/scripts/ before removing the directory, preserving custom hook logic Authored-By: Aneki (joshuaboys)
Review Feedback Addressed (round 2)Code Changes (1 commit pushed)
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 126cefd427
ℹ️ 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".
| # Update the updated timestamp | ||
| local today | ||
| today="$(date +%Y-%m-%d)" | ||
| sed -i "s/updated:.*/updated: \"$today\"/" "$target/.aps/config.yml" |
There was a problem hiding this comment.
sed -i with portable edit logic
cmd_update_v2 uses sed -i without a backup suffix, which works on GNU sed but fails on BSD/macOS sed (macOS requires -i '' ...). Because this line runs on every v2 aps update, macOS users hit a hard failure and the update aborts under set -e; the same non-portable pattern is also used in migration/path-update code paths, so this should be fixed with a portable in-place edit helper.
Useful? React with 👍 / 👎.
Customization frontend for APS project setup — replaces shell-prompt wizard with granular Ratatui TUI wizard. Supports monorepo detection, per-tool config, template/path customization, and config-driven init. Decisions resolved: D-026 (cli/ in this repo), D-027 (git dep for eddacraft-tui), D-028 (port lint to Rust for shared parser). Authored-By: Aneki (joshuaboys)
- /plan now runs aps init if plans/ is missing - /plan checks plans/.aps-version against current (0.2.0), runs aps update if behind - aps init and aps update both write plans/.aps-version = 0.2.0
aps init now installs structure only (plans/, designs/, bin/aps, templates). Skill/harness setup is an explicit opt-in via: aps setup <harness> Supports claude-code, codex, opencode — TUI setup coming with TUI release. Fixes: skill files no longer appear in every repo by default.
- Redirect prompt_select/prompt_multi display output to stderr so command substitution captures only the selection value - Exit 1 (not 0) when cmd_migrate finds no v1 installation - Strengthen is_v1_layout to require ≥2 APS markers, preventing false positives on repos with unrelated bin/aps - Use cp -a designs/. to include dotfiles during migration Authored-By: Aneki (joshuaboys)
1 participant
Summary
aps initv2 with shell-prompt wizard,aps migratefor v1→v2 conversion,aps updatev2-aware refresh, multi-tool support for 6 targets,config.ymlgeneration,aps-rules-v2.mdandproject-context.mdtemplates.Test plan
aps init --non-interactiveproduces correct.aps/layout with config.ymlaps migrate --dry-runpreviews v1→v2 changes without modifying filesaps migrateconverts v1 layout cleanly (old dirs removed, hooks rewritten, backup created)build.shgenerates all 14 agent files, is idempotent🤖 Generated with Claude Code