Skip to content

Commit 52259ea

Browse files
Point Codex verify-specs at the simulator-broker slug
Why: The verify-specs Codex action still wrote session artifacts under agent-harness/simulator-broker-app after the harness product slug became simulator-broker. Changed: - .codex/environments/environment.toml: verify-specs session-dir uses agent-harness/simulator-broker/manual-verify. - spec/agents.md and spec/build-and-test.md: Codex harness actions must use the same product slug, not package.json name simulator-broker-app. Verification: - npm run agent:verify -- --profile spec-only --paths .codex/environments/environment.toml spec/agents.md spec/build-and-test.md --session-dir task-sessions/verify-specs-slug-20260819: pass. Affected: - .codex/environments/environment.toml - spec/agents.md - spec/build-and-test.md Refs: - 34d3755 (Point agent session dirs at the simulator-broker slug) - adfc9a8 (Hardcode the harness product slug as simulator-broker) Session: - task-sessions/verify-specs-slug-20260819 Co-authored-by: Cursor <cursoragent@cursor.com>
1 parent 34d3755 commit 52259ea

3 files changed

Lines changed: 3 additions & 2 deletions

File tree

.codex/environments/environment.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,6 @@ icon = "test"
2929
command = '''
3030
#!/usr/bin/env bash
3131
set -euo pipefail
32-
npm run agent:verify -- --profile spec-only --paths spec/README.md --session-dir "${AGENT_HOME:-$HOME/.agents}/agent-harness/simulator-broker-app/manual-verify"
32+
npm run agent:verify -- --profile spec-only --paths spec/README.md --session-dir "${AGENT_HOME:-$HOME/.agents}/agent-harness/simulator-broker/manual-verify"
3333
'''
3434
platform = "darwin"

spec/agents.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,7 @@ Maintainers and agent runs still use `agent:context` / `agent:verify` /
4040

4141
- `script/build_and_run.sh` is the canonical macOS app kill/build/run entrypoint for local work.
4242
- `.codex/environments/environment.toml` must expose a `Run` action that points at `./script/build_and_run.sh`.
43+
- Codex harness actions in `.codex/environments/environment.toml`, including `verify-specs`, must pass `--session-dir` under `${AGENT_HOME:-$HOME/.agents}/agent-harness/simulator-broker/`. The product slug is `simulator-broker`; it is not derived from `package.json` name `simulator-broker-app`.
4344
- App workflow changes should route through the repo-shipped skills above instead of ad hoc shell chains or undocumented local commands.
4445
- When the app needs desktop-only behavior that SwiftUI cannot express cleanly, document the exact gap first and keep the implementation covered by `swiftui-pro` and `xcode-build`.
4546

spec/build-and-test.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -242,7 +242,7 @@ npm run agent:complete -- --session-dir "${AGENT_HOME:-$HOME/.agents}/agent-harn
242242
## Verification obligations
243243

244244
- `npm run agent:context -- --paths <files> --session-dir <dir>` is the routing and obligation source of truth for a task.
245-
- Task session dirs live under `${AGENT_HOME:-$HOME/.agents}/agent-harness/simulator-broker/`. The product slug is `simulator-broker`; it is not derived from `package.json` name `simulator-broker-app` or from the checkout directory. `CODEX_HOME` remains a compatibility parent for legacy session roots.
245+
- Task session dirs live under `${AGENT_HOME:-$HOME/.agents}/agent-harness/simulator-broker/`. The product slug is `simulator-broker`; it is not derived from `package.json` name `simulator-broker-app` or from the checkout directory. Codex harness actions in `.codex/environments/environment.toml`, including `verify-specs`, must use this same product slug. `CODEX_HOME` remains a compatibility parent for legacy session roots.
246246
- Implementation-owned paths require the `implementation` profile, which runs
247247
the complete `npm test` suite plus the `agent-harness` TypeScript build and
248248
Vitest suite. Specs, manifests, and harness contract paths require

0 commit comments

Comments
 (0)