Skip to content

Integrate thinking orbs for enhanced loading animations - #633

Open
harshil-2096 wants to merge 1 commit into
mainfrom
feat/thinking-orb-loader
Open

Integrate thinking orbs for enhanced loading animations#633
harshil-2096 wants to merge 1 commit 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
Cosmetic loading UX and a small new UI dependency; no changes to auth, APIs, or data handling.

Overview
Replaces plain “Loading application…” text and the Spinner in server initialization with the thinking-orbs ThinkingOrb animation (connecting state) during session checks, app boot, and while the TrueFoundry agent UI server resolves.

Adds thinking-orbs@^0.3.1 to frontend and @truefoundry/trueforge-ui, introduces a shared Loader in App.tsx, and updates ServerInitLoader accessibility (aria-label="Loading" on the status region). Tests mock the package and assert the orb appears during pending truefoundry server init. Patch changeset entries cover @truefoundry/trueforge and @truefoundry/trueforge-ui.

Reviewed by Cursor Bugbot for commit 59a1577. 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: 59a1577

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.

Cursor Bugbot has reviewed your changes using default effort and found 2 potential issues.

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 59a1577. Configure here.

Comment thread packages/frontend/src/App.tsx
>
<Spinner size={28} className="text-text-primary" />
<span className="sr-only">Loading</span>
<ThinkingOrb state="connecting" size={64} speed={1} theme="auto" paused={false} aria-hidden />

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Unthemed orb can lose contrast

Low Severity

ServerInitLoader pins ThinkingOrb to theme="auto", which falls back to prefers-color-scheme when no ancestor has data-theme or a dark/light class. The same loader is the withRouter Suspense fallback outside ThemeProvider, so a dark OS preference on a light host page can render light ink on a light background.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 59a1577. Configure here.

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