mirror of
https://github.com/AutoMaker-Org/automaker.git
synced 2026-02-04 09:13:08 +00:00
feat: remove "Agent Tools" from side navigation and related components
- Deleted the "Agent Tools" navigation item from the sidebar. - Updated keyboard shortcuts and app store types to reflect the removal. - Cleaned up imports and references in relevant files. - Retained the `agent-tools-view.tsx` component for potential future use.
This commit is contained in:
@@ -1,29 +0,0 @@
|
||||
## Summary
|
||||
|
||||
I've implemented the requested feature. Here's what was changed:
|
||||
|
||||
### Changes Made to `apps/app/src/components/ui/course-promo-badge.tsx`:
|
||||
|
||||
1. **Removed the modal dialog completely** - The `Dialog` component and all its sub-components (`DialogContent`, `DialogHeader`, `DialogTitle`, `DialogDescription`, `DialogFooter`) along with the modal state (`open`) have been removed.
|
||||
|
||||
2. **Changed from button to direct link** - The `<button>` element that triggered the modal has been replaced with an `<a>` anchor tag that links directly to `https://agenticjumpstart.com`.
|
||||
|
||||
3. **Simplified imports** - Removed unused imports:
|
||||
- Removed: `Rocket`, `ExternalLink`, `Code`, `MessageSquare`, `Brain`, `Terminal` from lucide-react
|
||||
- Removed: All Dialog-related imports from `./dialog`
|
||||
- Removed: `Button` import
|
||||
- Kept: `Sparkles` and `X` (still needed for the badge UI)
|
||||
|
||||
4. **Added proper link attributes** - Added `target="_blank"` to open in a new tab and `rel="noopener noreferrer"` for security best practices.
|
||||
|
||||
5. **Updated dismiss button** - Added `e.preventDefault()` to the dismiss button's click handler to prevent the link from being followed when clicking the X to dismiss the badge.
|
||||
|
||||
### Behavior:
|
||||
- **Before**: Clicking "Become a 10x Dev" opened a modal dialog with course information and a "Get Started" button that opened the site
|
||||
- **After**: Clicking "Become a 10x Dev" navigates directly to `https://agenticjumpstart.com` in a new tab
|
||||
- The dismiss (X) button functionality remains unchanged - clicking it hides the badge
|
||||
|
||||
### Notes for Developer:
|
||||
- The TypeScript error shown during build check is unrelated to this change (it's in `use-auto-mode.ts` file and was pre-existing)
|
||||
- The component is now significantly simpler with less code to maintain
|
||||
- No tests were found specifically for this component, so no test updates were needed
|
||||
@@ -1,14 +0,0 @@
|
||||
{
|
||||
"category": "Uncategorized",
|
||||
"description": "instead of showing a modal when clicking become a 10x dev, just load up a new link directly to the agenticjumpstart.com site",
|
||||
"steps": [],
|
||||
"status": "completed",
|
||||
"images": [],
|
||||
"imagePaths": [],
|
||||
"skipTests": true,
|
||||
"model": "opus",
|
||||
"thinkingLevel": "none",
|
||||
"id": "feature-1765524702667-c4ede0vih",
|
||||
"startedAt": "2025-12-12T07:41:40.120Z",
|
||||
"updatedAt": "2025-12-12T07:43:00.682Z"
|
||||
}
|
||||
@@ -0,0 +1,33 @@
|
||||
The navigation now shows only:
|
||||
- **Project** section: Kanban Board, Agent Runner
|
||||
- **Tools** section: Spec Editor, Context, AI Profiles
|
||||
|
||||
The "Agent Tools" item has been successfully removed from the side navigation.
|
||||
|
||||
## Summary
|
||||
|
||||
I've successfully removed "Agent Tools" from the side navigation by making the following changes:
|
||||
|
||||
### Files Modified:
|
||||
|
||||
1. **`apps/app/src/components/layout/sidebar.tsx`**
|
||||
- Removed the navigation item for "Agent Tools" (id: "tools")
|
||||
- Removed the unused `Wrench` icon import from lucide-react
|
||||
|
||||
2. **`apps/app/src/components/ui/keyboard-map.tsx`**
|
||||
- Removed `tools: "Agent Tools"` from `SHORTCUT_LABELS`
|
||||
- Removed `tools: "navigation"` from `SHORTCUT_CATEGORIES`
|
||||
|
||||
3. **`apps/app/src/store/app-store.ts`**
|
||||
- Removed `"tools"` from the `ViewMode` type union
|
||||
- Removed `tools: string` from the `KeyboardShortcuts` interface
|
||||
- Removed `tools: "T"` from `DEFAULT_KEYBOARD_SHORTCUTS`
|
||||
|
||||
4. **`apps/app/src/app/page.tsx`**
|
||||
- Removed the import for `AgentToolsView`
|
||||
- Removed the `case "tools": return <AgentToolsView />` from the view switch statement
|
||||
|
||||
### Notes:
|
||||
- The `agent-tools-view.tsx` component file was **not** deleted, as the user only requested to remove it from the navigation. The component remains in the codebase in case it's needed later.
|
||||
- The keyboard shortcut "T" is now available for other features if needed.
|
||||
- All lint errors found are pre-existing in the codebase and not related to these changes.
|
||||
@@ -0,0 +1,14 @@
|
||||
{
|
||||
"category": "Uncategorized",
|
||||
"description": "for now, just remove agent tools from the side navigation, they are useless",
|
||||
"steps": [],
|
||||
"status": "completed",
|
||||
"images": [],
|
||||
"imagePaths": [],
|
||||
"skipTests": true,
|
||||
"model": "opus",
|
||||
"thinkingLevel": "none",
|
||||
"id": "feature-1765525566442-wsdrtodkc",
|
||||
"startedAt": "2025-12-12T07:46:08.507Z",
|
||||
"updatedAt": "2025-12-12T07:48:34.967Z"
|
||||
}
|
||||
Reference in New Issue
Block a user