fix: resolve merge conflicts and clean up expand project feature

Post-merge fixes for PR #36 (expand-project-with-ai):

- Fix syntax error in App.tsx Escape handler (missing `} else`)
- Fix missing closing brace in types.ts FeatureBulkCreateResponse
- Remove unused exception variables flagged by ruff (F841)
- Make nav buttons minimalist: remove text labels, keep icons + shortcuts
  - "Add Feature" → icon + N shortcut, tooltip "Add new feature"
  - "Expand" → icon + E shortcut, tooltip "Expand project with AI"

All checks pass: ruff, security tests, ESLint, TypeScript build.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
Auto
2026-01-10 10:50:28 +02:00
parent 0e7f8c657a
commit 1998de7c50
5 changed files with 12 additions and 11 deletions

View File

@@ -116,7 +116,7 @@ function App() {
if (e.key === 'Escape') {
if (showExpandProject) {
setShowExpandProject(false)
if (showSettings) {
} else if (showSettings) {
setShowSettings(false)
} else if (assistantOpen) {
setAssistantOpen(false)
@@ -174,10 +174,9 @@ function App() {
<button
onClick={() => setShowAddFeature(true)}
className="neo-btn neo-btn-primary text-sm"
title="Press N"
title="Add new feature"
>
<Plus size={18} />
Add Feature
<kbd className="ml-1.5 px-1.5 py-0.5 text-xs bg-black/20 rounded font-mono">
N
</kbd>
@@ -188,10 +187,9 @@ function App() {
<button
onClick={() => setShowExpandProject(true)}
className="neo-btn bg-[var(--color-neo-progress)] text-black text-sm"
title="Add multiple features via AI (Press E)"
title="Expand project with AI"
>
<Sparkles size={18} />
Expand
<kbd className="ml-1.5 px-1.5 py-0.5 text-xs bg-black/20 rounded font-mono">
E
</kbd>

View File

@@ -329,6 +329,9 @@ export interface FeatureBulkCreate {
export interface FeatureBulkCreateResponse {
created: number
features: Feature[]
}
// ============================================================================
// Settings Types
// ============================================================================