[#23] Token price display on story page

[realproject7]

Summary

• P5-1a: New lib/price.ts — reads current token price and reserve balance from MCV2_Bond using priceForNextMint and tokenBond view functions
• P5-1b: Story page header now displays token price and reserve balance in a 2-column info card
• All contract addresses imported from lib/contracts/constants.ts (no hardcoding)
• Reserve label adapts: WETH on testnet, $PLOT on mainnet

Fixes #23

Test plan

• Verify price card renders on story pages with bonded tokens
• Verify graceful fallback (no price card) for tokens without a bond
• Verify testnet shows "WETH", mainnet shows "$PLOT"
• npm run lint and npm run typecheck pass

🤖 Generated with Claude Code

[project7-interns]

LGTM. Verified:

• MCV2_BOND and IS_TESTNET correctly imported from constants.ts (no hardcoded addresses)
• Reserve label adapts correctly (WETH/testnet, $PLOT/mainnet)
• Server-side price fetch with graceful null fallback (no price card when bond doesn't exist)
• Parallel multicall for priceForNextMint + tokenBond is efficient
• ABI destructuring correctly skips to reserveToken and reserveBalance fields

Minor note: pricePerToken/reserveBalance could display long decimal strings (e.g. "0.000000000000001234") — consider truncating to 4-6 significant digits in a follow-up for cleaner UX. Not blocking.

Approved.

[project7-interns]

Verdict: REQUEST CHANGES

Summary

The PR adds a token price card and uses the shared contract constants correctly, but it does not satisfy the issue acceptance criteria for the second field on the story page.

Findings

• [medium] Issue #23 explicitly calls for displaying the current price and the total supply minted, but the new utility only reads reserveBalance from tokenBond() and the story page renders that as Reserve instead. That means the required minted-supply value is still missing, so the PR does not fully implement P5-1b.
  • File: lib/price.ts:27
  • Suggestion: Extend the on-chain read path to fetch the token's minted supply from the appropriate contract/view, return it from TokenPriceInfo, and render that value on the story page instead of reserve balance.

Decision

Requesting changes because the story page currently shows the wrong metric for the second data point required by issue #23.

[project7-interns]

LGTM. Correctly replaced reserve balance with ERC-20 totalSupply on the storyline token — matches issue #23 spec. ABI simplified, interface cleaned up, UI label updated to "Supply Minted". Approved.

[project7-interns]

Verdict: APPROVE

Summary

The story page now shows the required token price and minted supply metrics, and the implementation still uses the shared contract constants and testnet/mainnet reserve label handling correctly. lint-and-typecheck is passing.

Findings

• none

Decision

Approving because the PR now satisfies issue #23 without the earlier metric mismatch.