Commit Graph

6 Commits

Author SHA1 Message Date
Andy fe7290a850 V2.7.0 (#100)
* feat(memory): replace FalkorDB with LadybugDB embedded database

Remove Docker dependency for Graphiti memory integration by switching
to LadybugDB, an embedded graph database that works via monkeypatch
with graphiti-core's KuzuDriver.

Changes:
- Remove all FalkorDB configuration and connection code
- Simplify config to use GRAPHITI_DB_PATH for local storage
- Update requirements to use real_ladybug + graphiti-core
- Update documentation for Python 3.12+ requirement

This makes the memory system much simpler to set up - no Docker needed.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* chore: remove docker-compose.yml (FalkorDB no longer used)

FalkorDB has been replaced with LadybugDB embedded database,
so Docker is no longer required for the memory integration.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* feat: add automated release workflow with code signing

- Add release.yml workflow triggered on version tags (v*)
- Support manual dry-run builds via workflow_dispatch
- Build for macOS (arm64 + x64), Windows, and Linux
- macOS code signing with Developer ID certificate
- macOS notarization support for Gatekeeper approval
- Add entitlements.mac.plist for hardened runtime
- Generate SHA256 checksums for all release artifacts
- Auto-generate changelog from PR labels
- Add pnpm caching to CI workflow for faster builds
- Add lint, typecheck, and build steps to CI
- Update package.json with artifactName for consistent naming
- Fix extraResources filter to exclude test venvs

Artifacts will be named: Auto-Claude-{version}-{platform}-{arch}.{ext}

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* refactor: replace Docker/FalkorDB with embedded LadybugDB for memory system

This major refactoring eliminates the Docker dependency for the memory system
by switching from FalkorDB to LadybugDB (embedded graph database).

Backend changes:
- Add query_memory.py: Python CLI for memory queries via subprocess
- Add kuzu_driver_patched.py: Patched Kuzu driver with FTS index support
- Add ollama_model_detector.py: Auto-detect Ollama embedding models
- Update client.py to use patched driver for proper FTS functionality
- Fix parameter handling for LadybugDB compatibility

Frontend changes:
- Add memory-service.ts: Node.js service wrapping Python subprocess
- Add memory-handlers.ts: IPC handlers for memory operations
- Add api-validation-service.ts: Validate LLM/embedder API keys
- Remove docker-service.ts and falkordb-service.ts (no longer needed)
- Update MemoryBackendSection with multi-provider embedder support
- Update InfrastructureStatus to show LadybugDB status
- Simplify GraphitiStep onboarding (no Docker setup required)

Key improvements:
- Zero Docker dependency - fully embedded database
- Multi-provider embedder support (OpenAI, Gemini, Voyage, Ollama, Azure)
- Hybrid RAG with semantic search, FTS, and graph traversal
- Automatic FTS index creation via patched driver
- Python 3.12 compatibility (LadybugDB requirement)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix: correct model name and release workflow conditionals

Bug fixes:
- Fix model ID: claude-sonnet-4-5-latest → claude-sonnet-4-5-20250514
  (using dated version for stability)
- Fix release workflow: add github.event_name checks before accessing
  inputs.dry_run to prevent errors on tag push events

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* feat: add Ollama embedding model support with auto-detected dimensions

- Add known model lookup table for popular Ollama embedding models:
  - embeddinggemma (768 dim) - Google's lightweight model
  - qwen3-embedding:0.6b/4b/8b (1024/2560/4096 dim) - Qwen3 series
  - nomic-embed-text, mxbai-embed-large, bge-large, all-minilm
- Auto-detect embedding dimensions for known models (no need to set OLLAMA_EMBEDDING_DIM)
- Fix config validation to not require OLLAMA_EMBEDDING_DIM for known models
- Fix PatchedKuzuDriver to set _database attribute (required by Graphiti)
- Fix get_status_summary to use db_path instead of deprecated falkordb_host

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* refactor: rebrand memory system UI and simplify configuration

- Rename "Graphiti" to "Memory" throughout UI
- Remove LLM provider selection (Claude SDK handles RAG)
- Keep embedding provider selection (OpenAI, Ollama, Voyage, Google, Azure)
- Add Ollama model pull/download support
- Change default storage path from ~/.auto-claude/graphs to ~/.auto-claude/memories
- Make embedding provider fields conditional based on selection
- Add OllamaModelSelector component with auto-detection
- Create simplified MemoryStep for onboarding wizard
- Update SecuritySettings with provider-specific field rendering

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix(ui): replace Unix shell syntax with cross-platform git commands

Fixes #90

Worktrees were showing "0" for all file changes on Windows because
Unix shell constructs (`2>/dev/null || echo`) are invalid in cmd.exe.

Changes:
- Replace `2>/dev/null || echo` with TypeScript try-catch
- Add `stdio: ['pipe', 'pipe', 'pipe']` to capture stderr
- Fix 7 locations in worktree-handlers.ts:
  - git rev-list --count (2 locations)
  - git diff --stat
  - git diff --numstat
  - git diff --name-status
  - git diff --shortstat
  - git merge-base --is-ancestor

The error "The system cannot find the path specified" was caused by
cmd.exe trying to interpret `/dev/null` as a literal file path.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

* fix(cli): update graphiti status display for LadybugDB

After migrating from FalkorDB (Docker-based) to LadybugDB (embedded),
the validate_environment function tried to access 'host' and 'port'
keys that no longer exist in the graphiti status dictionary.

Changes:
- Replace host:port display with db_path for embedded database
- Use .get() for safe key access

This fixes a KeyError crash when running auto-claude builds with
Graphiti memory enabled.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

* fix: improve Ollama UX in memory settings

- Replace manual Ollama config with OllamaModelSelector component
- Remove Base URL field (auto-detected from Ollama)
- Remove manual embedding dimension input (auto-detected from model)
- Show only installed models as selectable options
- Add download buttons for recommended models not yet installed
- Make embeddinggemma the recommended default model
- Remove qwen3-embedding from recommendations (focus on quality models)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* chore: update CI and release workflows, remove changelog config

- Removed the obsolete changelog configuration file.
- Updated CI workflow to use `pnpm run test` instead of `pnpm test`.
- Modified release workflow to use `pnpm install --frozen-lockfile` for consistent dependency installation.
- Enhanced artifact handling by validating the presence of build artifacts before proceeding.

These changes streamline the workflows and improve reliability in the build process.

* fix: resolve all CI failures in PR #100

Python lint fixes (ruff):
- Fix import sorting in kuzu_driver_patched.py, ollama_model_detector.py, query_memory.py
- Add noqa: F401 for kuzu import used only for availability check

Frontend:
- Update pnpm-lock.yaml to remove ioredis dependencies

Test fixes:
- Update test_graphiti.py to reflect new multi-provider architecture
- Embedder is now optional (keyword search fallback works)
- LLM provider validation removed (Claude SDK handles RAG)
- Fix FalkorDB references to LadybugDB (db_path instead of host/port)

Config consistency:
- Fix get_graphiti_status() to be consistent with is_valid()
- Embedder errors are now warnings, not blockers

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* refactor: update memory test suite for LadybugDB

Replace FalkorDB-based tests with LadybugDB (embedded database) tests:

- test_ladybugdb_connection() - Verify embedded DB works
- test_save_episode() - Save test data to graph
- test_keyword_search() - Keyword fallback (no embeddings needed)
- test_semantic_search() - Vector search with embeddings
- test_ollama_embeddings() - Direct Ollama embedding test
- test_graphiti_memory_class() - Full wrapper class test
- test_database_contents() - Debug view of DB contents

Usage:
  python integrations/graphiti/test_graphiti_memory.py --test ollama
  python integrations/graphiti/test_graphiti_memory.py --test connection
  python integrations/graphiti/test_graphiti_memory.py  # all tests

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix: resolve remaining CI failures

Python:
- Add missing blank line after imports in query_memory.py (ruff I001)

TypeScript:
- Fix mock getBestAvailableProfile signature to accept optional parameter

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix: remove f-string prefixes from strings without placeholders

Fixes ruff F541 errors in test_graphiti_memory.py

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* style: apply ruff formatting to 4 files

Auto-formatted kuzu_driver_patched.py and test_graphiti_memory.py

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix: address CodeRabbit review comments from PR #100

- release.yml: Fix changelog output property (outputs.body not outputs.changelog)
- config.py: Update Anthropic model to generic claude-sonnet-4-5 identifier
- api-validation-service.ts: Fix error message to include both sk- and sess- prefixes
- ci.yml: Add quotes around command substitution for safety
- OllamaModelSelector.tsx: Add error logging to catch block
- OllamaModelSelector.tsx: Add AbortController cleanup pattern for unmount
- SecuritySettings.tsx: Add useEffect to sync showOpenAIKey prop changes
- SecuritySettings.tsx: Fix stale state in toggleShowApiKey
- SecuritySettings.tsx: Add aria-labels to password visibility buttons

Verified: nomic-embed-text uses 768 dimensions (CodeRabbit was incorrect about 1024)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* docs: add CodeRabbit review response tracking

Document rejected CodeRabbit suggestions with justification:
- nomic-embed-text uses 768 dims (not 1024 as claimed)
- MemoryStep checkmark UX is intentional design

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix: sort imports in memory.py for ruff I001

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix: address additional CodeRabbit review comments

Security fixes:
- kuzu_driver_patched.py: Add try/finally blocks for connection cleanup
- query_memory.py: Use parameterized queries to prevent Cypher injection
- query_memory.py: Use public get_validation_errors() instead of private method
- release.yml: Use ./* glob pattern to prevent option injection

Code quality:
- ollama_model_detector.py: Fix type annotation (str | None)
- cleanup-version-branches.sh: Fix empty tag count edge case
- config.py: Remove unused _validate_llm_provider method

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix: update test and apply ruff formatting

- Remove test assertion for deleted _validate_llm_provider method
- Apply ruff format to query_memory.py

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix: address CodeRabbit review feedback

- kuzu_driver_patched.py: Simplify multi-line f-string to single line
- config.py: Clarify LadybugDB requires Python 3.12+ in docstrings
- ci.yml: Fix pnpm store path echo command quoting
- test_graphiti.py: Use public API get_validation_errors() instead of private method

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* feat: add embedding provider change detection and fix import ordering

Graphiti improvements:
- Add provider change detection in GraphitiMemory.initialize()
- Warn users when embedding provider changes to prevent dimension mismatches
- Guide users to run migration script or reset state
- Add test_provider_naming.py demo script for provider-specific database naming

Code quality fixes:
- Move inline `import re` to module top in query_memory.py
- Alphabetically order standard library imports (PEP8 compliance)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix(ci): quote GITHUB_OUTPUT for shell safety

Quote the $GITHUB_OUTPUT variable in the pnpm store path
echo command to safely handle paths containing spaces.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix: add GH_TOKEN and homepage for release workflow

- Add GH_TOKEN to all package steps (required for electron-builder to download native prebuilds)
- Add homepage and repository fields to package.json (required for Linux builds)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix: add author email for Linux builds

electron-builder FPM target requires author email for .deb packages

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix: make macOS notarization optional

Notarization can fail due to certificate issues - don't block the build.
Unsigned DMGs still work, users just see a security warning.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix: save notarization logs to private artifact instead of public logs

- Captures Apple notarization failure details in downloadable artifact
- Keeps sensitive paths and info out of public CI logs
- Only repo collaborators can access the notarization-logs artifact

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix: respect user's memory enabled flag in query_memory CLI

Remove the forced `config.enabled = True` override that ignored user's
explicit disable choice. The CLI tool should respect the enabled flag -
callers using the semantic-search command are explicitly requesting
memory functionality.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix: improve migrate_embeddings robustness and correctness

- Add proper cleanup of source client when target initialization fails
- Fix database name derivation to use source provider's signature
  instead of incorrectly using current config's signature
- Add validation to prevent no-op migration between same providers
- Remove unused imports (json, Optional, GraphitiState)
- Fix ruff formatting issues (line length)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* style: fix ruff linting errors in graphiti queries

- Remove f-string prefix from strings without placeholders
- Fix line length formatting issues

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix: use shell guard for notarization credentials check

The step-level `if: env.APPLE_ID != ''` condition was evaluated before
the step's env block was available, causing notarization to always be
skipped. Replace with a runtime shell guard that checks the environment
variable when the step actually runs.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix: allow @lydell/node-pty build scripts in pnpm v10

pnpm v10 blocks dependency lifecycle scripts by default. Add
@lydell/node-pty to onlyBuiltDependencies array so its native
module build scripts can run during installation.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* feat(ui): add project tab bar from PR #101

Cherry-picked from PR #101 for testing:
- Add tab state management to project store
- Create ProjectTabBar and SortableProjectTab components
- Remove project dropdown from Sidebar
- Add drag-and-drop tab reordering
- Include unit tests for tab functionality

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* chore: simplify notarization step after successful setup

Remove debug logging and notarization-logs artifact upload now that
Developer ID Application certificate is properly configured.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix: check APPLE_ID in shell instead of workflow if condition

secrets context cannot be used directly in if expressions.
Check env var in shell script instead.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix(ui): change agent profile fallback from 'Balanced' to 'Auto (Optimized)'

The getProfileDisplay() function in AgentProfileSelector had a defensive
fallback that incorrectly displayed "Balanced" when no profile was found.
Changed to display "Auto (Optimized)" to match the actual default profile
configured in DEFAULT_APP_SETTINGS.

This ensures consistency with the intended default behavior where fresh
users should see "Auto (Optimized)" as their agent profile selection.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* refactor(ui): simplify reference files and images handling in task modal

Remove separate reference files and images sections from the new task modal.
Images are now displayed as small clickable thumbnails directly below the
description field, and files can be referenced via @mentions in the description.

- Remove "Reference Images" toggle and dedicated ImageUpload section
- Remove "Referenced Files" section and ReferencedFilesSection component usage
- Add inline thumbnail display (64x64px) below description with remove buttons
- Update hint text to clarify drag & drop and paste functionality
- Clean up unused imports and state variables

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* chore: use GitHub noreply email for author field

Prevents spam while maintaining traceability for open source project.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix: create Python venv in userData for packaged apps

On Linux AppImages, the bundled resources directory is read-only,
which prevented venv creation. Now packaged apps store the venv in
userData (~/.config/auto-claude-ui on Linux) which is always writable.

This follows XDG conventions and fixes the Arch Linux AppImage issue.

Fixes: venv creation failure on Linux AppImage

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* feat(ui): add keyboard shortcuts and tooltips for project tabs

Add standard browser/editor-style keyboard shortcuts for tab navigation:
- Cmd/Ctrl+1-9 to switch to specific tabs
- Cmd/Ctrl+Tab/Shift+Tab to cycle through tabs
- Cmd/Ctrl+W to close current tab

Replace native title tooltips with Radix tooltips that:
- Show after 200ms instead of ~1s native delay
- Display shortcuts in styled kbd badges
- Match app design with smooth animations

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* feat: improve task creation UX with @ autocomplete and better drag-drop

Fixes three UX issues reported by users:

1. Add @ file autocomplete in task description
   - Type @ to see file suggestions
   - Filters files as you type
   - Keyboard navigation (arrows, Enter, Escape)
   - Shows file path for disambiguation

2. Fix file browser scroll in project explorer
   - Remove conflicting ScrollArea wrapper
   - Let virtualizer handle scrolling directly
   - Files now visible after expanding deep folders

3. Add auto-scroll during drag-and-drop
   - Scroll form container when dragging near edges
   - Makes it possible to drag files to textarea when scrolled out of view

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* feat(agent): enhance task restart functionality with new profile support

- Updated `restartTask` method to accept an optional `newProfileId` parameter, allowing for profile swapping during task restarts.
- Added logic to set the active profile if a new profile ID is provided.
- Adjusted event handling for `auto-swap-restart-task` to pass the new profile ID.

fix(agent): correct source type in rate limit info for spec creation

- Changed source type from 'task' to 'roadmap' in `createSDKRateLimitInfo` calls for better clarity in spec creation context.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

* fix(ui): fix tab persistence and scope terminal shortcuts

Two fixes:

1. Tab persistence on restart: Added explicit handling for when no tabs
   are open after app restart. Now auto-opens the first project tab
   instead of showing empty tab bar.

2. Keyboard shortcut scoping: Cmd/Ctrl+T now behaves contextually:
   - On Agent Terminals view: Opens new terminal (existing behavior)
   - On other views (Kanban, etc.): Opens Add Project dialog

   Added isActive prop to TerminalGrid to scope shortcuts to when
   the terminal view is active.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix: unify default database path to ~/.auto-claude/memories

The default path was inconsistent across files:
- utils.ts used 'graphs'
- memory-service.ts used 'memories'
- .env.example documented 'graphs'

Unified all to use 'memories' to match Python backend config.py.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* auto-claude: subtask-1-1 - Add projectPath prop to PreviewPanel and implement custom img component

- Added projectPath prop to PreviewPanel interface
- Implemented custom img component for ReactMarkdown that converts relative paths
  (like .github/assets/...) to file:// URLs for Electron
- Updated ChangelogDetails to get selected project and pass its path to PreviewPanel
- Images now display correctly in preview mode while markdown source remains unchanged

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix(merge): use stored baseBranch from task metadata for merge operations

Previously, merge logic hardcoded 'main' as the comparison branch when
detecting what files changed in a task. This broke the workflow where:
- Tasks branch FROM the configured default (main) or user-specified branch
- Tasks merge INTO the user's current working branch

Now the merge system:
1. Reads baseBranch from task_metadata.json (set during task creation)
2. Passes --base-branch to Python CLI for merge and merge-preview
3. Uses this for refresh_from_git() comparisons throughout the merge pipeline
4. Falls back to auto-detection (main/master/develop) if not specified

Files modified across frontend (TypeScript) and backend (Python) to thread
the task_source_branch parameter through the entire merge flow.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix(merge): increase AI merge timeout from 2 to 10 minutes

Large merge operations with many conflicting files were timing out
before completion. The AI merge resolution was processing files
successfully but hitting the 2-minute limit when handling 17+ files.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix: use venv Python for terminal name generation

TerminalNameGenerator was using system Python which doesn't have
claude_agent_sdk installed. Now uses pythonEnvManager to get the
venv Python path where dependencies are installed.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix: prevent task title from blocking edit/close buttons

Two issues were blocking the edit/close buttons in TaskDetailModal:

1. The title element was extending beyond its visual boundaries
   - Added overflow-hidden to container and truncate to title

2. The Electron window's draggable region was capturing mouse events
   - Added electron-no-drag class to the button container to allow
     normal mouse interactions in that area

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix: address CodeRabbit review issues

- memory-service.ts: Add app.getAppPath() path resolution for packaged
  Electron apps. This ensures query_memory.py is found in both dev and
  production builds, consistent with title-generator.ts pattern.

- MemoryStep.tsx: Make CheckCircle2 icon conditional on kuzuAvailable
  state. Previously showed success checkmark even when database wasn't
  available, misleading users.

- GitHubSetupModal.tsx: Fix incorrect API method name from
  getStoredGitHubToken to getGitHubToken. Also adds existing auth
  detection to skip already-completed authentication steps.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* Readme for installors

* Project tab persistence and github org init on project creation

* fix(ui): address CodeRabbit PR review issues

Fix actual bugs identified in PR #100 review:
- Fix race condition in memory-handlers.ts timeout handling by using
  single timeout with proper cleanup and resolved flag
- Fix API key resolution in GraphitiStep.tsx to handle groq, azure_openai,
  and ollama providers
- Add TabState interface and getTabState/saveTabState to ElectronAPI types
- Add mock implementations for browser development

Also includes:
- Exclude pnpm-lock.yaml from check-yaml (uses URLs with colons as keys)
- Auto-fixes from pre-commit hooks (trailing whitespace, EOF, ruff-format)

Note: nomic-embed-text dimension (768) and import ordering were verified
as correct - CodeRabbit suggestions were false positives.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix(tests): update tab management tests for IPC-based persistence

The project store now uses IPC (saveTabState/getTabState) instead of
localStorage for tab state persistence. Updated tests to:

- Remove localStorage.setItem assertions (no longer used)
- Update restoreTabState test to reflect it's now a no-op (actual
  loading happens via loadProjects → getTabState)
- Add getTabState/saveTabState mocks to test setup

This fixes the CI test failures where tests expected localStorage
calls but the implementation uses IPC.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-22 14:17:08 +01:00
AndyMik90 03ccce5cc1 feat: add required GitHub setup flow after Auto Claude initialization
This ensures users properly configure GitHub before using Auto Claude,
which is necessary for the branch-based workflow to function correctly.

Changes:
- Add GitHubSetupModal component with 3-step flow:
  1. GitHub OAuth authentication (via gh CLI)
  2. Auto-detect repository from git remote
  3. Select base branch for task worktrees (with recommended default)
- Add IPC handlers for detectGitHubRepo and getGitHubBranches
- Integrate modal into App.tsx to show after Auto Claude init
- Update ElectronAPI types and browser mocks

The flow now is:
1. User adds project
2. Git must be initialized (GitSetupModal if not)
3. Auto Claude initialized (creates .auto-claude folder)
4. GitHub setup required (new GitHubSetupModal)
5. Project ready for task creation

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-19 08:00:17 +01:00
AndyMik90 4c1dd89840 feat: add GitHub issue comment selection and fix auto-start bug
## Features
- Add comment selection UI when creating tasks from GitHub issues
  - Fetch and display all comments with author, timestamp, and preview
  - Allow users to select/deselect individual comments via checkboxes
  - Include "Select All" / "Deselect All" toggle functionality
  - Show selected comment count (e.g., "3/5 comments selected")
  - Only selected comments are included in the task description

- Fix auto-start bug where GitHub issues were immediately executed
  - Tasks now stay in "backlog" status after creation
  - Users must manually start tasks from the Kanban board

## Implementation Details

### Backend
- Added `getIssueComments` IPC handler to fetch comments separately
- Modified `investigateGitHubIssue` to accept selectedCommentIds parameter
- Updated comment filtering logic in buildIssueContext
- Removed automatic startSpecCreation call to prevent auto-execution

### Frontend
- Enhanced InvestigationDialog with scrollable comment list UI
- Updated GitHubAPI interface with getIssueComments method
- Modified hooks and stores to pass selected comment IDs through the chain
- Added projectId prop to InvestigationDialog for comment fetching

### Files Changed
- Backend handlers: investigation-handlers.ts, issue-handlers.ts, types.ts
- API layer: github-api.ts
- UI components: InvestigationDialog.tsx, GitHubIssues.tsx
- State management: github-store.ts, useGitHubInvestigation.ts
- Type definitions: types/index.ts, ipc.ts

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-18 18:27:01 +01:00
AndyMik90 7f3cd5969d feat: Add integrated release workflow with AI version suggestion
Adds a streamlined release workflow that uses AI (Claude Haiku) to analyze git commits and suggest semantic version bumps (major/minor/patch). Automatically updates package.json, commits, and pushes using a safe git workflow that preserves user's current work.

New features:

- RELEASE_SUGGEST_VERSION IPC for AI-powered version suggestions

- bumpVersion() with safe git workflow (stash/checkout/commit/restore)

- VersionSuggestion type with bumpType and reasoning

- Updated ReleaseProgress to include bumping_version stage

Also replaces VERSION file detection with requirements.txt across the codebase.
2025-12-18 10:32:55 +01:00
AndyMik90 afeb54fc2c New github oauth integration 2025-12-16 22:53:18 +01:00
AndyMik90 feb0d4e686 Refactoring for better codebase 2025-12-16 20:43:55 +01:00