Skip to content

Integrate thinking orbs for enhanced loading animations - #633

Open
harshil-2096 wants to merge 8 commits into
mainfrom
feat/thinking-orb-loader
Open

Integrate thinking orbs for enhanced loading animations#633
harshil-2096 wants to merge 8 commits into
mainfrom
feat/thinking-orb-loader

Conversation

@harshil-2096

@harshil-2096 harshil-2096 commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Summary

Screencast.from.2026-09-08.16-49-10.webm

Changes

How was this tested?

Checklist

  • I have read the contributing guidelines
  • pnpm build, pnpm test, pnpm typecheck, pnpm lint:ci, and pnpm format:check pass locally
  • Tests added/updated where it makes sense
  • No hand-edits to generated code (packages/trueforge-sdk, .github/fern/openapi/openapi.json, docs/openapi.json) — fork PRs omit SDK regen; maintainers regenerate after merge
  • Docs / .env.example updated if configuration or behavior changed

Note

Low Risk
Visual-only loading UX and dependency addition; no auth, data, or routing logic changes.

Overview
Boot and server-init loading states now show the thinking-orbs ThinkingOrb (connecting state) instead of CSS spinners or the primitive Spinner.

The host app adds a Loader that reads theme via useTheme so the orb matches light/dark inside ThemeProvider. ServerInitLoader in TrueForgeUIShell does the same with useThemeMode, which defaults to light when rendered outside ThemeProvider (e.g. Suspense). .boot-screen gets background: var(--primary-bg) so the orb stays visible against the pre-theme white body.

thinking-orbs is added as a dependency on @truefoundry/trueforge-ui and the frontend package; tests mock the orb and assert it appears during server init.

Reviewed by Cursor Bugbot for commit 033ef1c. Bugbot is set up for automated code reviews on this repo. Configure here.

@changeset-bot

changeset-bot Bot commented Sep 8, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 033ef1c

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@truefoundry/trueforge Patch
@truefoundry/trueforge-ui Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Stale Bugbot comment from a previous run.

Comment thread packages/frontend/src/App.tsx
Comment thread packages/trueforge-ui/src/containers/TrueForgeUIShell.tsx Outdated

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Stale Bugbot comment from a previous run.

Comment thread packages/trueforge-ui/src/containers/TrueForgeUIShell.tsx Outdated
Comment thread packages/trueforge-ui/src/containers/TrueForgeUIShell.tsx Outdated

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 9615373. Configure here.

Comment thread packages/frontend/src/App.tsx
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant