Skip to content

Keep coordination inbox payloads compact#244

Merged
NagyVikt merged 2 commits intomainfrom
agent/codex/make-colony-less-token-consuming-2026-04-29-02-54
Apr 29, 2026
Merged

Keep coordination inbox payloads compact#244
NagyVikt merged 2 commits intomainfrom
agent/codex/make-colony-less-token-consuming-2026-04-29-02-54

Conversation

@NagyVikt
Copy link
Copy Markdown
Collaborator

Automated by gx branch finish (PR flow).

NagyVikt added 2 commits April 29, 2026 02:57
The attention-budget test kept an older InboxMessage fixture shape after message action hints gained canonical reply and mark-read fields. Updating the fixture makes the hooks package typecheck meaningful again without changing runtime behavior.

Constraint: Production inbox payloads still expose both canonical action fields and the older reply_with/mark_read_with aliases.

Rejected: Cast the fixture to any | would hide future contract drift.

Confidence: high

Scope-risk: narrow

Directive: Keep test fixtures structurally complete when InboxMessage grows; avoid casts that bypass the contract.

Tested: pnpm --filter @colony/hooks typecheck; pnpm --filter @colony/hooks test -- attention-budget.test.ts; pnpm --filter @colony/hooks test; pnpm exec biome check packages/hooks/test/attention-budget.test.ts; git diff --check

Not-tested: repo-wide pnpm -r typecheck.
attention_inbox listed every stalled/dead lane by default, which made the normal coordination check expensive in active repos. This keeps the total count as the durable signal, caps returned lane rows, and lets MCP/CLI callers raise the row limit when they need the full list.

Constraint: Agents need stalled lane awareness without hydrating unbounded coordination history.

Rejected: Hide stalled lanes completely | loses takeover signal and makes the summary count unactionable.

Confidence: high

Scope-risk: narrow

Directive: Keep default coordination payloads bounded; add explicit expansion knobs instead of widening defaults.

Tested: pnpm --filter @colony/core test -- attention-inbox.test.ts; pnpm --filter @colony/mcp-server test -- server.test.ts; pnpm --filter @colony/hooks test -- attention-budget.test.ts; pnpm --filter @colony/core typecheck; pnpm --filter @colony/mcp-server typecheck; pnpm --filter @imdeadpool/colony-cli typecheck; pnpm exec biome check touched files; openspec validate agent-codex-make-colony-less-token-consuming-2026-04-29-02-54 --strict

Not-tested: full monorepo test suite
@NagyVikt NagyVikt merged commit b937fb7 into main Apr 29, 2026
@NagyVikt NagyVikt deleted the agent/codex/make-colony-less-token-consuming-2026-04-29-02-54 branch April 29, 2026 01:01
NagyVikt added a commit that referenced this pull request Apr 29, 2026
The compact-inbox implementation merged in PR #244, but the source OpenSpec task file still had unchecked completion boxes. Recording the PR merge and cleanup evidence keeps the durable task artifact aligned with repo truth.

Constraint: Follow-up is bookkeeping only; runtime code already merged in b937fb7.

Rejected: Leave the checklist stale | future agents would treat completed work as unfinished.

Confidence: high

Scope-risk: narrow

Directive: Do not reopen compact inbox implementation from this checklist; use PR #244 merge evidence as the closeout source.

Tested: openspec validate agent-codex-make-colony-less-token-consuming-2026-04-29-02-54 --strict; git diff --check

Not-tested: runtime suites not rerun; no runtime code changed.

Co-authored-by: NagyVikt <nagy.viktordp@gmail.com>
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