Skip to content

content: publish latest to production (src/content → peanut-content@72e1f53)#2420

Closed
0xkkonrad wants to merge 1 commit into
mainfrom
content/publish-to-main-202607151140
Closed

content: publish latest to production (src/content → peanut-content@72e1f53)#2420
0xkkonrad wants to merge 1 commit into
mainfrom
content/publish-to-main-202607151140

Conversation

@0xkkonrad

Copy link
Copy Markdown
Contributor

Bumps src/content on production (main) from f3c0ef672e1f53 (peanut-content latest = mono@3ce8f73). Single-file submodule pointer change publishing content already merged to mono. The automated update-content PRs reach this same pointer but target dev, which never promotes.

…2e1f53)

Bumps src/content on production (main) from f3c0ef6 → 72e1f53
(peanut-content latest = mono@3ce8f73). Single-file submodule
pointer change. Publishes content already merged + mirrored from mono that the
dev-targeted auto-PRs never promote to main.
@vercel

vercel Bot commented Jul 15, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
peanut-wallet Ready Ready Preview, Comment Jul 15, 2026 11:46am

Request Review

@0xkkonrad 0xkkonrad enabled auto-merge July 15, 2026 11:40
@coderabbitai

coderabbitai Bot commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Walkthrough

The src/content subproject pointer is updated from commit f3c0ef6... to 72e1f53....

Changes

Content submodule update

Layer / File(s) Summary
Update content subproject pointer
src/content
The pinned src/content commit changes to 72e1f53....

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

Suggested labels: enhancement

Suggested reviewers: hugo0, chip-peanut-bot[bot]

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the production content publish and the submodule target commit.
Description check ✅ Passed The description matches the change by explaining the src/content submodule bump to main and why it promotes content.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

Copy link
Copy Markdown
Contributor

🧪 UI test report — ✅ all green

Suites

  • unit: 1836 ran, 0 failed, 0 skipped, 29.8s

📊 Coverage (unit)

metric %
statements 58.2%
branches 42.2%
functions 47.0%
lines 58.3%
⏱ 10 slowest test cases
time test
4.1s src/components/Card/share-asset/__tests__/shareAssetLayout.test.ts › never places two stickers in heavy overlap (broad seed sweep)
0.5s src/components/Card/share-asset/__tests__/shareAssetLayout.test.ts › every sticker stays within canvas at any count
0.3s src/app/actions/__tests__/api-headers.test.ts › should include Content-Type in updateUserById
0.3s src/app/actions/__tests__/api-headers-extended.test.ts › should not include apiKey in updateUserById body
0.2s src/components/Card/share-asset/__tests__/shareAssetLayout.test.ts › keeps stickers off the username pill (final pass respects the keep-out)
0.2s src/lib/validation/token.test.ts › should resolve ETH on different chains
0.1s src/app/(mobile-ui)/qr-pay/__tests__/qr-pay-states.test.tsx › Manteca PIX form ready shows merchant card + amount input + pay button
0.1s src/components/Global/GeneralRecipientInput/__tests__/GeneralRecipientInput.test.tsx › should handle valid 9-digit US account
0.1s src/components/Global/GeneralRecipientInput/__tests__/GeneralRecipientInput.test.tsx › should handle valid US account with spaces 2
0.1s src/components/Global/GeneralRecipientInput/__tests__/GeneralRecipientInput.test.tsx › should handle invalid ETH address (missing 0x prefix)
📍 Inline annotations are in the **Unit test report** check above. Coverage artifact: `coverage-unit`. Generated by `.github/workflows/tests.yml`.

@github-actions

Copy link
Copy Markdown
Contributor

Code-analysis diff

Painscore total: 6122.15 → 6122.15 (0)
Findings: 0 net (+0 new, -0 resolved)

@coderabbitai coderabbitai Bot added the enhancement New feature or request label Jul 15, 2026

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@src/content`:
- Line 1: Resolve the broken English content-route references reported by
validate-links in the content submodule: fix or remove each invalid link, then
update the submodule pointer to a corrected published commit that passes
validation. Do not merge the current content pointer while link validation still
reports failures.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 8c1394ca-d8b9-4e59-8a63-6dd150cdd285

📥 Commits

Reviewing files that changed from the base of the PR and between 9b4f6f2 and 5d59d43.

📒 Files selected for processing (1)
  • src/content

Comment thread src/content
@@ -1 +1 @@
Subproject commit f3c0ef661f1e890250de7892419018c0a0af59a7
Subproject commit 72e1f5327edddae0831443cd56ccec7b3f66af7d

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🎯 Functional Correctness | 🟠 Major | ⚡ Quick win

Do not merge this content pointer while link validation is failing.

The target commit causes validate-links to report 12 broken links to missing English content routes. Fix or remove the invalid references in the content submodule, publish a corrected commit, or pin this repository to a commit that passes validation.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@src/content` at line 1, Resolve the broken English content-route references
reported by validate-links in the content submodule: fix or remove each invalid
link, then update the submodule pointer to a corrected published commit that
passes validation. Do not merge the current content pointer while link
validation still reports failures.

Source: Pipeline failures

@0xkkonrad

Copy link
Copy Markdown
Contributor Author

Superseded: main's src/content pointer moved under this PR (#2423 card-contact hotfix), making it conflicting and stale. A fresh publish PR at the latest peanut-content commit (which contains both this content and the hotfix) replaces it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants