Skip to content

Expose Colony claim edit trace identity#1

Merged
NagyVikt merged 1 commit into
devfrom
agent/codex/audit-omx-colony-claim-edit-trace-dev-2026-04-29-13-31
Apr 30, 2026
Merged

Expose Colony claim edit trace identity#1
NagyVikt merged 1 commit into
devfrom
agent/codex/audit-omx-colony-claim-edit-trace-dev-2026-04-29-13-31

Conversation

@NagyVikt
Copy link
Copy Markdown
Collaborator

Summary

  • add OMX_COLONY_TRACE=1 JSONL trace records for Colony claim/edit correlation
  • carry trace_id, tool_use_id, file_path(s), session, cwd, repo_root, branch, timestamp, source through Colony bridge envelopes
  • classify mcp__colony__task_claim_file PostToolUse as claim telemetry and edit/write/multiedit/patch transports with compact file path extraction

Verification

  • npm run build
  • node --test dist/colony/tests/bridge.test.js
  • node --test --test-name-pattern 'bridges PreToolUse Edit payloads to Colony before the edit proceeds' dist/scripts/tests/codex-native-hook.test.js
  • npm run check:no-unused
  • npm run lint -- src/colony/bridge.ts src/colony/tests/bridge.test.ts

Risk

  • Diagnostic-only trace writes; normal hook decisions stay unchanged.

OMX forwards Codex hook payloads to Colony, but claim and edit rows can only correlate if both sides expose the same compact identity fields. This adds a trace id derived from Codex tool_use_id when available, records normalized file paths for edit and patch transports, and writes OMX_COLONY_TRACE=1 JSONL rows for claim/edit comparison without changing normal hook behavior.

Constraint: Diagnostic trace must not include full patch or edit bodies.

Rejected: Add more agent prompt nudges | claims already exist, the defect is correlation identity.

Confidence: high

Scope-risk: narrow

Directive: Keep trace rows compact and audit-only; do not count PostToolUse recovery as pre-edit success.

Tested: npm run build; node --test dist/colony/__tests__/bridge.test.js; node --test --test-name-pattern 'bridges PreToolUse Edit payloads to Colony before the edit proceeds' dist/scripts/__tests__/codex-native-hook.test.js; npm run check:no-unused; npm run lint -- src/colony/bridge.ts src/colony/__tests__/bridge.test.ts

Not-tested: live Colony database correlation against production sessions
@NagyVikt NagyVikt force-pushed the agent/codex/audit-omx-colony-claim-edit-trace-dev-2026-04-29-13-31 branch from a2dfda1 to b419b53 Compare April 29, 2026 11:47
@github-actions github-actions Bot added size/L and removed size/XL labels Apr 29, 2026
@NagyVikt NagyVikt merged commit 3eb8392 into dev Apr 30, 2026
16 of 20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant