Task: Remove gates for fully shipped features

Usage: /routine-shipped-feature-inliner [package-or-path]

Find a flag whose decision has already been made everywhere, prove it is
forgotten rather than deliberate, inline the path it always takes, delete the
other branch and the flag definition, and ship via /ship.

Contract

Read .agents/skills/routine-shared/contract.md first — preconditions,
scope resolution, verify gates, ship protocol, budget, universal STOPs. Floor
even without it: never ship red; one finding per ship; anything in a
DO_NOT_DO.md is a hard STOP.

Workflow

1. Scope

Resolve scope per the contract.

2. Discover

Grep the package for gate shapes:

• mobile/src/config.ts → config.features.* compile-time constants
• process.env.* used as a boolean switch; values seeded in .env.example
  and deploy configs (deploy/, root bex.yaml)
• is*Enabled / enable* booleans, route guards, early returns on a config
  value

3. Prove it — deliberate gate vs forgotten flag (mandatory triage)

A flag is only a finding when all of these hold:

1. It holds one value in every environment: grep every setter and reader,
   .env.example, deploy configs — no environment or code path flips it.
2. Nothing marks it deliberate: check root .pm/DO_NOT_DO.md, the package's
   .pm/DO_NOT_DO.md, docs/adrs/, and code comments at the definition.
3. git log on its introduction and last value change shows it has sat at its
   final value for a while, with no sign of a pending decision.

Any evidence of intent → not a finding; note it as skipped.

4. Fix

Inline the path the flag always takes. Delete: the now-dead branch, the flag
definition, its plumbing (props/params that only carried it), and tests that
only exercised the removed branch. Update tests that asserted the flag's value
to assert the now-unconditional behavior. Never change the flag's value on
the way out — inline only flags already at their final value.

5. Verify

Owning package's full gate per the contract. If the flag guarded UI, verify the
surviving path still renders (mobile: light and dark per mobile/AGENTS.md).

6. Ship

Compose /ship for this one flag. Loop within budget.

What NOT to do

• NEVER inline, remove, or flip config.features.agentChat. The mobile
  agent-chat surface stays gated off by owner decision
  (mobile/.pm/DO_NOT_DO.md) — it is a deliberate gate, permanently out of
  scope.
• The backend featureFlags GraphQL field is a cross-package contract
  (backend-cluster/backend-v2/src/features/ledger/api/resolvers/ledger-legacy-resolver.query.ts,
  consumed by the mobile app). Removing or reshaping it is a coordinated
  multi-package change — report it, don't fix it (universal STOP).
• Never delete unflagged dead code — that is routine-dead-code-removal's job.
• Never "finish shipping" a feature by turning its flag on. Flipping a value is
  a product decision, not maintenance.

Arguments

$ARGUMENTS