[#661] Fix hardcoded Sepolia addresses in claim and status

[realproject7]

Summary

• claim and status commands imported MCV2_BOND_ADDRESS and STORY_FACTORY_ADDRESS directly from constants, which always resolved to Sepolia addresses — causing reverts on mainnet
• Made storyFactory and mcv2Bond public readonly on the PlotLink client class (already resolves based on chainId)
• Updated both commands to use client.mcv2Bond / client.storyFactory instead of hardcoded imports

Changed files

• packages/cli/src/sdk/client.ts — storyFactory and mcv2Bond changed from private to public readonly
• packages/cli/src/commands/claim.ts — use client.mcv2Bond instead of MCV2_BOND_ADDRESS
• packages/cli/src/commands/status.ts — use client.mcv2Bond and client.storyFactory instead of hardcoded constants

Test plan

• npm run build passes
• npm run typecheck passes
• plotlink claim resolves correct Bond address on mainnet (verified during E2E in [#320] Fix backfill: missing title, silent data loss, cron auth #322)
• plotlink status uses correct StoryFactory and Bond addresses per chain

Fixes #661

🤖 Generated with Claude Code

[vercel]

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

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
plotlink	Ignored		Mar 31, 2026 8:21am

[project7-interns]

T2b APPROVE

Perfect +7/-7 net-zero fix:

1. Root cause correct — claim and status imported hardcoded Sepolia constants instead of using the chain-aware client properties.
2. Visibility change is appropriate — storyFactory and mcv2Bond promoted from private to public readonly. These are immutable after construction and already chain-resolved, so safe to expose. erc8004Registry stays private — intentional scoping.
3. Dead imports removed — MCV2_BOND_ADDRESS and STORY_FACTORY_ADDRESS no longer imported in the command files.
4. No behavioral change beyond the fix — same code paths, just using the right addresses.

[project7-interns]

Verdict: APPROVE

Summary

The change is minimal and fixes the command paths that were bypassing the client's chain-aware contract addresses. claim and status now read the active Bond and StoryFactory addresses from the PlotLink client instead of hardcoded Sepolia constants.

Findings

• None blocking.

Decision

Approving because the implementation matches #661, keeps the fix scoped, and routes the affected commands through the existing chain-aware address resolution.