refactor(seller): remove the dead second agent axis (#200)

[orveth]

Removes [seller_roster] and seller_node::roster — the dead second agent axis described in #200.

Why it is safe to delete rather than finish

Nothing read it. The run loop resolves harnesses through boot_agent_registry → seller_agents::resolve(seller, &home.config.agents) into an AgentRegistry; select_agent(), OfferNeeds and SelectedAgent had no consumers outside their own module, and SellerRosterConfig / RosterAgent appeared only in home.rs, where they were parsed, validated and serialised. An operator could write a full [seller_roster] section — agents, capabilities, min_rate_sats, slots — have it validate cleanly, and get no behavioural change whatsoever.

Under cleancut, a config surface a user can set that does nothing is a relic, and v1 ships without relics.

Verification

• The compiler is the control on "nothing consumed it." grep suggested zero consumers; a clean build after deletion proves it. cargo build and cargo test both exit 0 with --features acp,wallet,gateway: 646 passed, 0 failed, 3 ignored, 0 filtered out.
• ⚠ That 0 filtered out is load-bearing, not decoration. The wallet feature gates the entire seller_node module and mobee-core has default = [], so a plain cargo test compiles none of these tests and still prints ok — over a zero numerator. Reviewers re-running this should use the same feature set and check that the count moved.
• Test-count delta is exactly the removed file's own tests: #[test] in mobee-core goes 534 → 528, and the deleted roster.rs contained exactly 6. No other test was touched.
• No deployed config breaks. MobeeConfig carries deny_unknown_fields, so removing the field would make any config still carrying the section fail to load. Scanned every config.toml under /srv/forge and ~/.config — 34 configs, 0 with [seller_roster].
• Pure deletion: no behavioural lines added.

Scope

Deliberately its own PR. The roster-drop feature (#254) extends seller_agents::AgentRegistry and is a separate semantic unit — a removal buried in a feature diff would spoil both reviews.

Closes #200.

🤖 Generated with Claude Code

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
mobee	Ready	Preview	Jul 30, 2026 8:31am