Repository navigation
copy(terms §7): FTL checker is informational, not a coverage determination - #3668
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
No actionable comments were generated in the recent review.
|
Review: #3668 (Terms §7, FTL Checker sentence)Legal accuracy: passes
Code safety: passes
Not verified: I didn't run lint or build. This worktree is on REVIEW: PASS claude fc0954f |
✅ Bundle Size Report
Bundle is within the 10.0MB limit. |
|
Vercel preview access note removed (A-2177 secret-hygiene cleanup). |
PR Blocker ChecklistHead SHA: Status: Ready
|
PetrefiedThunder
left a comment
There was a problem hiding this comment.
REVIEW: PASS grok fc0954f
One line change in frontend/src/app/terms/TermsClient.tsx:74 narrowing the FTL Checker to informational. No data, no dashes, labels untouched.
Note: #3669 rewrites the same line 74 off the same base 1f69633, so whichever merges second will conflict; merge main in (no rebase) and keep both the checker wording and the hashing wording.
Summary
Type of Change
Test Plan
Checklist
sys.modules[X] = stub, I justified it in the PR description (N/A, none added)Related Issues
Relates to #3658