chore(platformctl): remove dead agent_access/_types.py (imported nowhere) #769

New issue

Closed

opened 2026-06-08 23:19:52 +02:00 by ollama · 0 comments

ollama commented

2026-06-08 23:19:52 +02:00

Collaborator

Spec sources (whitelist)

control-plane/platformctl/agent_access/_types.py — dead model code, imported nowhere
policies/agent-capabilities.yaml — loaded by NO production code
control-plane/platformctl/cli.py line ~366 — DEFAULT_POLICY with always-empty allowlist
state/strategy/autonomy-tiered-execution-design-2026-06-02.md — design doc claiming present-tense this IS the live Tier-0 allowlist

Extracted context

The autonomy capability catalog (#566) is loaded by NO code. The production path uses DEFAULT_POLICY with an always-empty allowlist. agent_access/_types.py is imported nowhere — dead model code, self-labeled "Not yet wired to runtime."

The autonomy design doc claims (present-tense) it IS the live Tier-0 allowlist — false. This misleads agents reading the doc.

This is separate from the architectural decision (#733 item A, operator-gated) about whether to wire or demote the whole capability catalog. This issue only removes the dead-code distraction.

Scope

Delete control-plane/platformctl/agent_access/_types.py (imported nowhere)
Add # Design-target / not-yet-shipped. See #733 for operator decision on wire vs demote. comment above DEFAULT_POLICY in cli.py:366
Verify no imports break: rg 'agent_access._types' control-plane/ should return 0

Do NOT:

Delete policies/agent-capabilities.yaml (it's documentation, may be used later)
Modify autonomy-tiered-execution-design doc (already updated in #748)
Make any architectural decision (that's #733)

Acceptance criteria

agent_access/_types.py removed
0 references to deleted module remain
DEFAULT_POLICY site has clear design-target comment
715+ tests still green

Do NOT read

Full repo — only the 4 files listed above

Agent notes

Recommended executor: Gemini 3.5 Flash
Size: Trivial (~5 LOC + 1 file delete)
Review tier: tier/trivial
Audit ref: state/audit/deepseek-2026-06-08-multiperspective.md §E6

## Spec sources (whitelist) - `control-plane/platformctl/agent_access/_types.py` — dead model code, imported nowhere - `policies/agent-capabilities.yaml` — loaded by NO production code - `control-plane/platformctl/cli.py` line ~366 — `DEFAULT_POLICY` with always-empty allowlist - `state/strategy/autonomy-tiered-execution-design-2026-06-02.md` — design doc claiming present-tense this IS the live Tier-0 allowlist ## Extracted context The autonomy capability catalog (#566) is loaded by **NO code**. The production path uses `DEFAULT_POLICY` with an **always-empty allowlist**. `agent_access/_types.py` is imported nowhere — dead model code, self-labeled "Not yet wired to runtime." The autonomy design doc claims (present-tense) it IS the live Tier-0 allowlist — false. This misleads agents reading the doc. This is separate from the architectural decision (#733 item A, operator-gated) about whether to wire or demote the whole capability catalog. This issue only removes the dead-code distraction. ## Scope 1. Delete `control-plane/platformctl/agent_access/_types.py` (imported nowhere) 2. Add `# Design-target / not-yet-shipped. See #733 for operator decision on wire vs demote.` comment above `DEFAULT_POLICY` in `cli.py:366` 3. Verify no imports break: `rg 'agent_access._types' control-plane/` should return 0 Do NOT: - Delete `policies/agent-capabilities.yaml` (it's documentation, may be used later) - Modify `autonomy-tiered-execution-design` doc (already updated in #748) - Make any architectural decision (that's #733) ## Acceptance criteria - [ ] `agent_access/_types.py` removed - [ ] 0 references to deleted module remain - [ ] `DEFAULT_POLICY` site has clear design-target comment - [ ] 715+ tests still green ## Do NOT read - Full repo — only the 4 files listed above ## Agent notes - Recommended executor: Gemini 3.5 Flash - Size: Trivial (~5 LOC + 1 file delete) - Review tier: tier/trivial - Audit ref: state/audit/deepseek-2026-06-08-multiperspective.md §E6