Repository navigation
fix(editor): mount the new-company human check on the editor route - #371
Merged
important-new merged 1 commit intoOct 12, 2026
Merged
Conversation
…reen editor The editor route is declared outside the workspace layout, so the provider that raises the human check was never mounted there: publishing from the editor in a new company's first 24 hours showed no check, and the report mail it queued was never vouched for, so the delivery pass held it until the window ends. The editor already fetches the session context for its preview link, so its loader now also returns the open cooling window and site key, the page mounts the provider, and the publish dialog takes the unlock time from a prop (it read it from a session context the editor does not have, so its new-company notice never showed there). Tests cover the loader, the dialog prop and the EditorHumanCheck component; that the route uses it is checked on the live deployment rather than by a unit test. The provider mount lives in a small EditorHumanCheck component (with its own test), so the route only gains a thin default-export wrapper, the import and one prop. File-size baseline for the editor route raised by 12 lines for exactly that wiring. chrome-allow: the check and the notice only appear for a SaaS company inside its first 24 hours, which a standalone dev server cannot produce; the fix is verified against the live SaaS after deploy Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01C8qkG1Y2agL9r9s3CXziwF
important-new
changed the title
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
1 participant
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.
Summary
The full-screen editor route is declared outside the workspace layout, so the provider that raises the new-company human check was never mounted there. Publishing from the editor in a company's first 24 hours therefore showed no check, and the report mail it queued was never vouched for, so the delivery pass held it until the window ends. (Found while testing the check on a live deployment.)
EditorHumanCheckcomponent mounts the provider above the page; the route only gains a thin default-export wrapper.Test plan
EditorHumanCheck: a publish inside a window asks for the check; with no window it resolves at once.🤖 Generated with Claude Code
https://claude.ai/code/session_01C8qkG1Y2agL9r9s3CXziwF