fix(modules): remove default context-budget stamps #518

Merged

pdurlej merged 1 commit from codex/w9-agent-context-budget-cleanup into main

2026-05-27 08:59:51 +02:00

codex commented

2026-05-27 08:49:30 +02:00

Collaborator

Canary status: missing — this touches module manifests, so use required checks/canary before merge

Canary Context Pack

Product story

Agents should not read a uniform agent_context_budget: 1500 stamp as meaningful per-module metadata. If everything says the same thing, it is schema default, not signal.

What changed

Removed metadata.agent_context_budget: 1500 from module manifests where it only restated the schema default.
Kept non-default module budgets (1800) intact.
Clarified schema docs: specify agent_context_budget only for non-default budgets.

Why it changed

This rewrites the useful part of old #331 from current main: reduce manifest noise and remove an agent-misleading false signal.

Files touched

86 module manifests: one line removed from each default-budget module.
schema/module.schema.md: default/omit guidance clarified.

Relevant context

schema/module.schema.json already defines agent_context_budget.default = 1500.
Old #331 was stale/conflicting; this is the current-main rewrite.

Runtime evidence

None. Repo metadata only; no runtime mutation.

Known constraints

Broad mechanical diff by file count, but each touched manifest has the same one-line deletion. Non-default budgets remain visible.

Explicit out-of-scope

No schema JSON change, no module lifecycle/source/runtime metadata changes, no runbook changes.

Requested decision

Merge if checks are green and reviewers agree the default semantics are preserved.

Merge blockers

Any module validation failure, accidental removal of non-default budget, or hidden schema behavior relying on explicit default values.

Spec sources read

schema/module.schema.json — verified default agent_context_budget: 1500 exists.
schema/module.schema.md — updated human-readable default guidance.
modules/*/module.yaml — mechanical default-line cleanup.

Verification

rg -n "agent_context_budget: 1500" modules → 0 matches.
rg -n "agent_context_budget" modules → only two 1800 exceptions remain.
PYTHONPATH=control-plane python3 -m platformctl.cli validate all --json → exitCode 0.
git diff --check → clean.

Closes #331

Canary status: missing — this touches module manifests, so use required checks/canary before merge ## Canary Context Pack ### Product story Agents should not read a uniform `agent_context_budget: 1500` stamp as meaningful per-module metadata. If everything says the same thing, it is schema default, not signal. ### What changed - Removed `metadata.agent_context_budget: 1500` from module manifests where it only restated the schema default. - Kept non-default module budgets (`1800`) intact. - Clarified schema docs: specify `agent_context_budget` only for non-default budgets. ### Why it changed This rewrites the useful part of old #331 from current `main`: reduce manifest noise and remove an agent-misleading false signal. ### Files touched - 86 module manifests: one line removed from each default-budget module. - `schema/module.schema.md`: default/omit guidance clarified. ### Relevant context - `schema/module.schema.json` already defines `agent_context_budget.default = 1500`. - Old #331 was stale/conflicting; this is the current-main rewrite. ### Runtime evidence None. Repo metadata only; no runtime mutation. ### Known constraints Broad mechanical diff by file count, but each touched manifest has the same one-line deletion. Non-default budgets remain visible. ### Explicit out-of-scope No schema JSON change, no module lifecycle/source/runtime metadata changes, no runbook changes. ### Requested decision Merge if checks are green and reviewers agree the default semantics are preserved. ### Merge blockers Any module validation failure, accidental removal of non-default budget, or hidden schema behavior relying on explicit default values. ## Spec sources read - `schema/module.schema.json` — verified default `agent_context_budget: 1500` exists. - `schema/module.schema.md` — updated human-readable default guidance. - `modules/*/module.yaml` — mechanical default-line cleanup. ## Verification - `rg -n "agent_context_budget: 1500" modules` → 0 matches. - `rg -n "agent_context_budget" modules` → only two `1800` exceptions remain. - `PYTHONPATH=control-plane python3 -m platformctl.cli validate all --json` → exitCode 0. - `git diff --check` → clean. Closes #331

codex added 1 commit

2026-05-27 08:49:30 +02:00

fix(modules): remove default context-budget stamps

base-is-main / guard (pull_request) Successful in 2s

Details

canary-required / collect-diff (pull_request) Successful in 5s

Details

patchwarden-client-dry-run / collect-diff (pull_request) Successful in 5s

Details

python-ci / Python 3.11 (pull_request) Successful in 42s

Details

python-ci / Python 3.12 (pull_request) Successful in 43s