Skip to content

[#748] Revert CSP headers that broke Farcaster miniapp#751

Merged
realproject7 merged 1 commit intomainfrom
task/748-revert-csp
Apr 2, 2026
Merged

[#748] Revert CSP headers that broke Farcaster miniapp#751
realproject7 merged 1 commit intomainfrom
task/748-revert-csp

Conversation

@realproject7
Copy link
Copy Markdown
Owner

Summary

Fixes #748

Test plan

  • Farcaster miniapp loads correctly in Warpcast
  • No CSP-related errors in console
  • npm run build passes

🤖 Generated with Claude Code

Removes the entire headers() block from next.config.ts added in PR #747.
The CSP frame-ancestors directive was blocking Warpcast from framing
PlotLink as a miniapp.

Fixes #748

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 2, 2026

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

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
plotlink Ignored Ignored Apr 2, 2026 10:51am

Request Review

Copy link
Copy Markdown
Collaborator

@project7-interns project7-interns left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

T2b APPROVE — clean hotfix revert. Entire CSP headers() block removed, next.config.ts restored to empty NextConfig. Single file, -40 lines, no side effects. Unblocks Farcaster miniapp framing.

Copy link
Copy Markdown
Collaborator

@project7-interns project7-interns left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verdict: APPROVE

Summary

This matches the hotfix scope in issue #748. The PR cleanly removes the CSP headers() block from next.config.ts and restores the file to an empty config, with no unrelated changes.

Findings

  • None.

Decision

Approving because the revert is minimal, directly addresses the reported Farcaster breakage, and I do not see a remaining code-level mismatch against the ticket.

@realproject7 realproject7 merged commit f763385 into main Apr 2, 2026
5 checks passed
@realproject7 realproject7 deleted the task/748-revert-csp branch April 2, 2026 10:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

HOTFIX: Revert CSP headers — breaks Farcaster miniapp completely

2 participants