Skip to content

Stabilize React build setup and resolve key lint/type blockers#12

Open
DigitalBlueprint239 wants to merge 10 commits into
masterfrom
codex/audit-and-correct-errors-in-codebase
Open

Stabilize React build setup and resolve key lint/type blockers#12
DigitalBlueprint239 wants to merge 10 commits into
masterfrom
codex/audit-and-correct-errors-in-codebase

Conversation

@DigitalBlueprint239

Copy link
Copy Markdown
Owner

Motivation

  • Make the front-end build reproducible by resolving dependency conflicts, tooling misconfiguration and a set of deterministic lint/type errors blocking react-scripts build.

Description

  • Aligned and updated dev/runtime dependencies in package.json to resolve peer conflicts and add missing runtime packages (@supabase/supabase-js, recharts, tailwindcss etc.) and updated package-lock.json accordingly.
  • Switched PostCSS/Tailwind wiring to the CRA-compatible plugin (postcss.config.js now requires tailwindcss), and removed the old @tailwindcss/postcss usage.
  • Fixed DebugPanel hook-order/lint violations by avoiding early return on invalid props, adding safeResults fallbacks and guarding handlers with canRunAll/canTogglePassed in src/components/SmartPlaybook/DebugPanel.js.
  • Adjusted TypeScript setup by adding src/import-meta.d.ts (typing for import.meta.env) and updating tsconfig.json to modern module settings and selective excludes to avoid unrelated embedded subtrees from breaking the main compilation.
  • Small test/workflow fixes: updated src/App.test.tsx to be compatible with the TS/React setup.

Testing

  • Ran npm install --legacy-peer-deps — succeeded and produced an updated lockfile; vulnerability audit output noted but does not block build.
  • Installed @supabase/supabase-js and recharts separately with npm install --legacy-peer-deps — both succeeded.
  • Ran npm run build repeatedly and iteratively fixed blockers: the Tailwind/PostCSS plugin mismatch, React Hooks lint violations in DebugPanel, import.meta typing errors (fixed via src/import-meta.d.ts), and various missing deps; these fixes allowed the build to advance further than the initial failures.
  • Final npm run build still fails with an existing TypeScript prop error (a PlayerFeedback usage missing required drillId prop), which is outside the scope of the targeted fixes above and requires additional code cleanup; all other addressed issues succeeded.

Codex Task

@netlify

netlify Bot commented Feb 10, 2026

Copy link
Copy Markdown

Deploy Preview for magical-starlight-0c1207 failed.

Name Link
🔨 Latest commit c0db583
🔍 Latest deploy log https://app.netlify.com/projects/magical-starlight-0c1207/deploys/698b5397cbd9c60008994d2e

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant