Add Not Magic advisory config#1
Open
fentonbenjamin wants to merge 15 commits into
Open
Conversation
Badges → title → spine lines fade in sequentially, giving the output a composed-not-dumped feel. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Client was crashing at import time when OPENAI_API_KEY wasn't available during static page collection. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Users can pick gpt-4.1, local (no LLM), or auto (try LLM, fall back to local). Fallback reason shown in result badges. Enables side-by-side comparison of LLM vs heuristic shaping. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Three LLM options in dropdown. Any failure silently falls back to local heuristic engine with a note explaining why. No explicit local option in the picker — it's the safety net, not a choice. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Google OAuth + magic link sign-in (wow before login) - Save button on results — prompts sign-in if anonymous - Full canonical ShapeResult saved as JSONB - History page at /history with shape listing - RLS: users can only see their own shapes - Schema: shapes, collections, collection_items Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Use getSession() instead of getUser() to avoid hanging - Add catch/finally so loading always resolves - Client-only mount for UserMenu to prevent hydration mismatch - Tagline: "Paste text, get structured meaning." Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Gemini returns evidence as string instead of string[]. Coerce before Zod parse so all three models pass validation. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Lazy-load Supabase client in useEffect to avoid SSR/hydration failures that caused blank white screen in production. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Permalink pages at /s/[id] — shareable links for saved shapes - RLS: shapes publicly readable, owner-writable - Save returns permalink, "Copy link" button appears after save - Auto-save after sign-in if save was pending - Rich history page with spine preview and badges - Mobile-responsive: wrapping controls, smaller textarea, fluid layout - Supabase auth redirects fixed for production URL - listShapes now returns result JSONB for spine display Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Remove history from UserMenu (header button is enough) - Allow empty spine from model, fallback to title - Prevents "Too small: expected array >=1 items" validation error Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Not Magic — advisoryStable — all scopes resolved Resolved: testing, observability, ownership, rollback, general Show what this relies on · Report · Chain #15 · Advisory only. |
Owner
Author
|
Added an intentional Not Magic smoke conflict in docs/notmagic-smoke-conflict.md. The PR claims release readiness while declaring unresolved rollback ownership, muted observability, missing production-scale evidence, and no recovery owner. Expected advisory result: build safety may be relied on, release readiness should not be. |
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
Verification