[#246] Fix token symbol collision — redeploy with PLT- prefix

[realproject7]

Summary

• Changed StoryFactory token symbol prefix from PLOT- to PLT- in StoryFactory.sol to avoid MCV2_Bond duplicate symbol rejection
• Deployed new StoryFactory to Base Sepolia: 0xfa5489b6710Ba2f8406b37fA8f8c3018e51FA229
• Updated STORY_FACTORY in lib/contracts/constants.ts and packages/sdk/src/constants.ts
• All 15 forge tests pass
• No ABI changes

Deployment details

• Chain: Base Sepolia (84532)
• New contract: 0xfa5489b6710Ba2f8406b37fA8f8c3018e51FA229
• Old contract: 0x6B8d38af1773dd162Ebc6f4A8eb923F3c669605d

Operator action needed

• Update NEXT_PUBLIC_CONTRACT_ADDRESS env var if set
• Run migration to re-tag existing data if needed

Test plan

• forge test — 15/15 pass
• createStoryline on new contract succeeds without symbol collision

Fixes #246

🤖 Generated with Claude Code

[project7-interns]

Verdict: APPROVE

Summary

The app and SDK constants are updated consistently to the new Base Sepolia StoryFactory address 0xfa5489b6710Ba2f8406b37fA8f8c3018e51FA229, and I independently verified that the local plotlink-contracts workspace contains the corresponding PLT- symbol-prefix change in StoryFactory.sol.

Findings

• None in this PR. The repo-side updates are internally consistent and match the stated redeploy target.

Decision

Approving because the address rollout in plotlink is coherent and aligned with the contract change described in issue #246. CI was still pending at the time of review, and the deployment itself remains an external operator step.

[project7-interns]

Review: APPROVE

Checklist

Symbol prefix change (PLOT- -> PLT-): The Solidity change is not in this repo (contract source lives separately). The PR correctly focuses on updating the deployed address after redeployment. This is fine.

Contract address consistency: The new address 0xfa5489b6710Ba2f8406b37fA8f8c3018e51FA229 is updated in both:

• lib/contracts/constants.ts (web app, with env var override)
• packages/sdk/src/constants.ts (SDK)

The CLI (packages/cli/src/config.ts) imports from the SDK, so it picks up the change automatically. No other live code references the old address.

Old address in migrations: The old address 0x6B8d38af... still appears in supabase/migrations/00009_contract_address.sql and 00012_fix_contract_address_backfill.sql. These are historical migrations (backfilling existing data and fixing a prior tagging error) — they should NOT be modified. Correct as-is.

ABI changes: Confirmed — lib/contracts/abi.ts and packages/sdk/src/abi.ts are untouched in this PR. Symbol prefix is a constructor argument, not an ABI change.

Env var override: NEXT_PUBLIC_CONTRACT_ADDRESS env var override is preserved in the constants.ts pattern. PR description correctly notes operator action to update this env var.

Operator steps: PR mentions env var update and data re-tagging. Adequate.

No security concerns — this is a straightforward address swap after a contract redeploy with no ABI changes.