Repository navigation
fix: distinguish demo assumptions from workflow facts - #5
Merged
Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Yasuui
marked this pull request as ready for review
October 26, 2026 17:54
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. |
- MapperFace reads reduced motion through useSyncExternalStore and derives the blink instead of resetting state inside an effect. - WorkflowChat sends the intent per request, opens the brief from onFinish, and tracks streamed chunks during render instead of in an effect. - factsAndAssumptions de-duplicates evidence lines, so React keys stay unique and the brief no longer repeats a line. - Rendered-HTML tests now assert the current greeting and the data: URL download path that replaced blob URLs. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01DbmD6cAA88NEhhUnPW3ew9
The route parsed the raw body with req.json() and forwarded whatever arrived: any role (system included), any part type, remote file URLs and unbounded sizes. Malformed JSON surfaced as a 500. - lib/optimize-request.ts checks content type, body size (4 MB), message count, text length, file count, roles (user/assistant), part types, and inline data: attachments against one allowlist shared with the composer. Assistant history keeps only text parts. Rejections are 400/413/415 with a message the chat shows to the user. - The composer normalizes attachment media types, refuses unsupported images and caps PDFs and images at 2.5 MB before upload. - tests/optimize-request.test.mjs covers the UI's real payloads, injection of system turns, SSRF-style remote URLs, mislabelled binaries and size limits. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01DbmD6cAA88NEhhUnPW3ew9
- GitHub Actions runs lint, then tsc, next build and node --test, with no model key. - README states what is live (demo mode), what is built but off (live model), what the tests cover and what they do not. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01DbmD6cAA88NEhhUnPW3ew9
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01DbmD6cAA88NEhhUnPW3ew9
The earlier capture came from next dev and showed development issue badges. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01DbmD6cAA88NEhhUnPW3ew9
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
Yasuui
changed the title
Yasuui
changed the title
Yasuui
changed the title
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This merged change corrects the demo's evidence labels and workflow-step extraction, and adds request validation, lint fixes, CI and documentation. The later automation-plan work is tracked separately in #9.
This description was reconstructed after it was inadvertently replaced during the follow-up; it is not the original PR description.