Add content_type support for fiction vs cartoon storylines

[realproject7]

Summary

• Adds content_type column (fiction | cartoon, default fiction) to storylines table via migration
• /api/index/storyline accepts and validates contentType param, rejects invalid values with 400
• /api/storyline/update allows admin-only updates to content_type
• Story cards and detail page display a "Cartoon" badge for cartoon storylines
• Updated Supabase types, added CONTENT_TYPES constant, unit tests for badge and validation

Closes #1212

Test plan

• TypeScript typecheck passes
• ESLint passes (no new errors)
• Unit tests pass (StoryCard Cartoon badge, content-type validation)
• RE1/RE2 review: verify migration applies cleanly
• Verify default fiction for existing rows after migration
• Test indexing a cartoon storyline via API
• Test invalid contentType rejection (400)
• Test admin-only update restriction (403 for non-admin)

🤖 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		May 16, 2026 10:56pm

[realproject7]

RE2 Review: ✅ Approve

Clean implementation — migration is safe (NOT NULL DEFAULT won't break existing rows), API validation is correct with shared CONTENT_TYPES constant, admin-only gate on updates is properly enforced with 403, and badge rendering follows existing patterns.

Minor non-blocking note: no CHECK constraint on the column, so direct DB inserts could bypass validation. This is consistent with how genre/language are handled in the codebase, so not a blocker.

CI: lint-and-typecheck green, e2e pending at time of review.

[project7-interns]

Verdict: REQUEST CHANGES

Summary

The PR implements the main API/UI wiring for content_type, but the allowed value contract is not fully enforced. Invalid supplied values can still silently become fiction, and the database does not protect the foundational storylines.content_type column from out-of-band invalid writes.

Findings

• [medium] Invalid non-string/falsy contentType values are accepted as the default fiction instead of being rejected with 400.

  • File: src/app/api/index/storyline/route.ts:49
  • Suggestion: Validate based on property presence, not truthiness. For example, when contentType is present, require it to be a string and one of CONTENT_TYPES; only default to fiction when the field is absent.

• [medium] The migration adds content_type without a database constraint, so the stated allowed values are not enforced at the source of truth.

  • File: supabase/migrations/00039_add_content_type.sql:2
  • Suggestion: Add a CHECK constraint such as content_type IN ('fiction', 'cartoon'), ideally named, so admin SQL, service-role jobs, or future endpoints cannot persist unsupported content types.

Decision

Requesting changes because #1212 defines content_type as the foundation for downstream cartoon work. That field needs strict value enforcement before later tickets depend on it. Note: lint/typecheck is green, but e2e was still pending at review time.

[project7-interns]

Verdict: APPROVE

Summary

The requested fixes are in place. contentType is now rejected when present but non-string/invalid, and the migration adds a named CHECK constraint enforcing fiction/cartoon at the database layer.

Findings

• None blocking.

Decision

Approved. The prior RE1 findings are resolved. lint-and-typecheck is green on the updated commit; e2e was still pending at approval time and should be verified before merge.

[realproject7]

RE2 Re-review: ✅ Approve

Both fixes correctly address RE1's findings:

1. Strict validation — uses "contentType" in body for presence check, rejects non-string/invalid with 400, only defaults to fiction when field is absent.
2. CHECK constraint added to migration — DB-level enforcement now in place.

lint-and-typecheck green. e2e pending. No further concerns from RE2.