Uh oh!
There was an error while loading. Please reload this page.
refactor: consolidate Radix imports in the GitHub agent example - #388
refactor: consolidate Radix imports in the GitHub agent example#388lilienblum wants to merge 4 commits into
Conversation
Greptile SummaryThe PR migrates the GitHub agent dashboard’s two UI component sets from individual Radix packages to the consolidated
Confidence Score: 4/5The PR is not yet safe to merge because fresh dashboard installations can still resolve an unvalidated The documented installation runs Files Needing Attention: examples/github-agent/packages/dashboard/package.json and examples/github-agent/packages/dashboard/pnpm-lock.yaml
|
| 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
Uh oh!
There was an error while loading. Please reload this page.
Summary
radix-uipackageradix-uidependencyValidation
tsccheck for the migrated UI component setsgit diff --checkThe example's full typecheck still reports unrelated existing chart, API, and remote-agent dependency errors.
Linear: ALIEN-480