Fix site-breaking import bugs, untrack knowledge node_modules, add org-wide OpenRouter key - #27
Merged
Merged
Conversation
…rg-wide OpenRouter key A "split constants" refactor had stripped several imports/constants while leaving the JSX that used them (Logo, Mail, Phone, useState, FOUNDER_IMPACT, FREE_TOOLS_BLURB, PRICING_SECTIONS, Users/Calculator icons), which 500'd every page on the site including /admin, since the root layout renders <Footer/> on every route. Also consolidated a duplicate src/lib/config/social.ts / .ts+.tsx pair that had diverging data and caused non-deterministic import resolution. Also untracks packages/knowledge/node_modules (365 files, including a compiled esbuild binary) that got committed in the last PR because that package was missing the .gitignore every sibling package has. Wires a shared OPENROUTER_API_KEY through the existing org secrets pipeline (Doppler crewcircle-master/prod -> Pulumi ESC crewcircle/master -> GitHub org secret), matching how GITHUB_TOKEN/SENTRY_TOKEN/etc. are already distributed, plus an admin cost-tracking provider mirroring the existing Anthropic one. The actual secret value still needs to be set in Doppler by someone with access before it takes effect anywhere. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
6 tasks
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
2 participants
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
Logo,Mail,Phone,useState,FOUNDER_IMPACT,FREE_TOOLS_BLURB,PRICING_SECTIONS,Users/Calculatoricons) left behind by an earlier "split constants" refactor. These were 500-ing every page on the site, including/admin, since the root layout renders<Footer/>on every route.src/lib/config/social.ts/social.tsxpair that had diverging data (missing GitHub link, different Instagram URL) and caused non-deterministic import resolution depending on which file the bundler picked.packages/knowledge/node_modules/(365 files, including a compiledesbuildbinary) that got committed in the last merged PR because that package was missing the.gitignoreevery sibling package has. Added the missing.gitignore.OPENROUTER_API_KEYthrough the existing org secrets pipeline (Dopplercrewcircle-master/prod→ Pulumi ESCcrewcircle/master→ GitHub org secret), matching exactly howGITHUB_TOKEN/SENTRY_TOKEN/etc. are already distributed to every project. Also added an admin cost-tracking provider module mirroring the existing Anthropic one.afterEachimport inpackages/knowledge/test/internal/index.test.ts.Why
Verifying the admin console surfaced that the whole site was actually broken (500s), traced to leftover dead references from a prior refactor commit. Fixed those, then implemented the org-wide OpenRouter key request using the same secrets-distribution pattern already established for other shared credentials.
Reviewer notes
OPENROUTER_API_KEYsecret value still needs to be set in Doppler (doppler secrets set OPENROUTER_API_KEY=<key> --project crewcircle-master --config prod) by someone with access — this PR only wires the plumbing, it can't set the live secret.AppDeftype is missingbeta/featuredfields referenced byAppsGrid/AppCard, a few admin API routes return typedResponseinstead ofNextResponse, andprojects/[id]/page.tsximports aGithubicon that no longer exists in the installedlucide-reactversion.Test plan
npx tsc --noEmit— no errors (aside from the pre-existing items noted above)npx eslint src --quiet— clean/admin,/admin/costs,/admin/observability,/api/admin/*all return 200 with real content🤖 Generated with Claude Code