Skip to content

refactor: consolidate Radix imports in the GitHub agent example - #388

Draft
lilienblum wants to merge 4 commits into
mainfrom
dan/alien-480-standardize-dashboard-icons-and-consolidate-radix-imports
Draft

refactor: consolidate Radix imports in the GitHub agent example#388
lilienblum wants to merge 4 commits into
mainfrom
dan/alien-480-standardize-dashboard-icons-and-consolidate-radix-imports

Conversation

@lilienblum

Copy link
Copy Markdown
Contributor

Summary

  • migrate the GitHub agent dashboard's component primitives to the unified radix-ui package
  • update both existing UI component directories
  • replace 27 individual Radix dependencies with one radix-ui dependency

Validation

  • targeted tsc check for the migrated UI component sets
  • dependency and import audits
  • git diff --check

The example's full typecheck still reports unrelated existing chart, API, and remote-agent dependency errors.

Linear: ALIEN-480

@lilienblum lilienblum changed the title ALIEN-480: consolidate Radix imports in the GitHub agent example refactor: consolidate Radix imports in the GitHub agent example Aug 10, 2026
@lilienblum
lilienblum marked this pull request as ready for review August 10, 2026 15:49
@greptile-apps

greptile-apps Bot commented Aug 10, 2026

Copy link
Copy Markdown

Greptile Summary

The PR migrates the GitHub agent dashboard’s two UI component sets from individual Radix packages to the consolidated radix-ui package.

  • Rewrites primitive imports across 48 dashboard UI component files.
  • Removes the individual Radix dependencies and adds the consolidated package.
  • Replaces several other latest dependency declarations with compatible version ranges.

Confidence Score: 4/5

The PR is not yet safe to merge because fresh dashboard installations can still resolve an unvalidated radix-ui release.

The documented installation runs pnpm install, the committed lockfile has no package resolutions, and radix-ui remains declared with a caret range, so the dependency used by a fresh installation is not reproducible.

Files Needing Attention: examples/github-agent/packages/dashboard/package.json and examples/github-agent/packages/dashboard/pnpm-lock.yaml

Important Files Changed

Filename Overview
examples/github-agent/packages/dashboard/package.json Consolidates the Radix dependencies, but the compatible range and unresolved lockfile leave the previously reported reproducibility problem outstanding.
examples/github-agent/packages/dashboard/components/ui/form.tsx Migrates the Label and Slot imports to the corresponding consolidated-package namespaces while preserving component usage.
examples/github-agent/packages/dashboard/components/ui/sidebar.tsx Replaces direct primitive imports with consolidated Radix namespace imports without changing sidebar behavior.
examples/github-agent/packages/dashboard/ui/sidebar.tsx Applies the same consolidated import migration to the second dashboard UI component directory.

Reviews (3): Last reviewed commit: "fix(example): use compatible dependency ..." | Re-trigger Greptile

Comment thread examples/github-agent/packages/dashboard/package.json Outdated
@lilienblum
lilienblum marked this pull request as draft August 11, 2026 01:16
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