Skip to content

Conversation

@dhairyashiil
Copy link
Member

@dhairyashiil dhairyashiil commented Dec 27, 2025

Summary by cubic

Enable React Compiler in the companion app and update tooling and components to be compatible. This improves build setup and cleans up state handling and UI behaviors.

  • Dependencies

    • Enabled reactCompiler in app.json.
    • Configured Babel for react-compiler (panicThreshold: all_errors).
    • Added Metro resolver for react/compiler-runtime.
    • Added ESLint config with react-compiler plugin and new lint scripts; updated dev deps.
  • Refactors

    • Added "use no memo" in key screens, hooks, and context to align with React Compiler.
    • Switched Animated value to a ref-safe pattern in NetworkStatusBanner.
    • Normalized loading/saving state updates for consistent resets.
    • Simplified date/time formatting and removed unused code/comments.

Written for commit 5b37789. Summary will update automatically on new commits.

@dhairyashiil dhairyashiil requested a review from a team as a code owner December 27, 2025 13:36
@graphite-app graphite-app bot added the community Created by Linear-GitHub Sync label Dec 27, 2025
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 issue found across 23 files

Prompt for AI agents (all issues)

Check if these issues are valid — if so, understand the root cause of each and fix them.


<file name="companion/package.json">

<violation number="1" location="companion/package.json:45">
P2: Inconsistent package manager usage: This script uses `bun run` while all other chained scripts in this file use `npm run`. This could fail if bun is not installed and creates inconsistency in the codebase.</violation>
</file>

Reply to cubic to teach it or ask questions. Tag @cubic-dev-ai to re-run a review.

@vercel
Copy link

vercel bot commented Dec 27, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

3 Skipped Deployments
Project Deployment Review Updated (UTC)
cal Ignored Ignored Dec 27, 2025 4:29pm
cal-companion Ignored Ignored Preview Dec 27, 2025 4:29pm
cal-eu Ignored Ignored Dec 27, 2025 4:29pm

@dhairyashiil dhairyashiil marked this pull request as ready for review December 27, 2025 16:31
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 23 files

@volnei volnei merged commit 4036fb6 into main Dec 27, 2025
38 checks passed
@volnei volnei deleted the feat/companion-react-compiler branch December 27, 2025 19:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community Created by Linear-GitHub Sync ready-for-e2e size/XL

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants