feat: build Nvidia NIM AI assistant financial briefing drawer component - #116
Open
feyibosslady wants to merge 4 commits into
Open
feat: build Nvidia NIM AI assistant financial briefing drawer component#116feyibosslady wants to merge 4 commits into
feyibosslady wants to merge 4 commits into
Conversation
… summary - Create types for AssetBalance and WalletSummary - Add mock data with XLM, USDC, EURC, ASTRO tokens - Build AssetCard component showing ticker, balance, and USD equivalent - Build AssetBalanceGrid with responsive card grid layout - Handle loading and empty wallet states gracefully - Use design tokens from tokens.css with responsive grid - Export barrel for wallet feature Closes ASTROIDX556#61
…ules Closes ASTROIDX556#58 - Add simulator utility (src/features/policies/utils/simulator.ts) with pure evaluation logic for testing transactions against PolicyRule objects - Add PolicySandboxModal component with React Hook Form + Zod validated inputs for recipient address, amount, and asset type - Support multi-select rule picker with per-rule pass/fail evaluation - Display detailed violation messages and summary with compliance badge - Use existing Dialog component for accessible modal (focus trap, Escape, overlay dismissal) - Export new components from policies feature index
…nd search - Add AuditLogEntry type with Info/Warning/Critical severity levels in src/types/audit.ts - Create AuditLogStream component with real-time text search and severity dropdown filter - Render color-coded severity badges using design token variants (bg-info, bg-warning, bg-danger) - Include keyboard navigation (Escape to clear) and ARIA labels for screen reader accessibility - Provide 10 mock audit log entries covering agent actions, policy violations, and admin overrides Closes ASTROIDX556#60
- Create src/features/assistant/FinancialBriefingDrawer.tsx with Framer Motion animations - Implement chat state management with message history and loading spinners - Style chat bubbles and input controls using design tokens from src/styles/tokens.css - Add focus trapping for accessibility when drawer is open - Add preset prompt chips for common financial briefings (Daily Spend, Low Balance, Gas Fees, Treasury Report, Pending Approvals, Wallet Audit) - Integrate with existing AiBriefing data from useBriefing hook - Support streaming markdown chat responses with structured briefing widgets Closes ASTROIDX556#67
|
MergeKeeper review Scope: in scope for linked issue Implemented the Nvidia NIM AI assistant financial briefing drawer component and supporting files as specified in issue #67. Reviewed commit: |
|
@feyibosslady is attempting to deploy a commit to the Cjay's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
MergeKeeper merge status Status: blocked Reason: Next steps:
|
![mergekeeper[bot]](https://avatars.githubusercontent.com/u/173216192?s=60&v=4){kind=small}
There was a problem hiding this comment.
Approved
Implemented the Nvidia NIM AI assistant financial briefing drawer component and supporting files as specified in issue #67.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Implements the Nvidia NIM AI assistant financial briefing drawer component as specified in issue #67.
Changes
New file:
src/features/assistant/FinancialBriefingDrawer.tsx- Main drawer component with:AiBriefingdata fromuseBriefinghookNew file:
src/features/assistant/index.ts- Export barrel fileDesign & Styling
src/styles/tokens.css(colors, spacing, typography, shadows, border radius)bg-surface-secondary,border-border,text-foregroundfor assistant messagesbg-accent-gradientwithtext-background-secondaryfocus:border-gold,focus:ring-goldfor focus statesease-astroid(cubic-bezier(0.22, 1, 0.36, 1)) for consistent motionAccessibility
prefers-reduced-motionTesting
npm run typecheck- passes (pre-existing errors in unrelated transactions/page.tsx)npm run lint- passes for new filesnpm run build- compiles successfully (blocked by pre-existing issue in transactions/page.tsx)Closes #67