feat: Foundation - Storage + Config System (Issue #52) #55
Conversation
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
Move all 15 test files from co-located to __tests__ directories to align with convention used in mcp-server and subagents packages. Implementation: - Moved 15 test files to __tests__ directories across core package - Updated all imports to use relative paths from __tests__ directories - Fixed scanner test path calculation for new location - Removed all old co-located test files - Fixed linting errors (unused imports and variables) Testing: - All 15 test files reorganized - Imports updated and verified - No linting errors - Path calculations corrected Files reorganized: - storage/__tests__/ (2 tests) - events/__tests__/ (1 test) - __tests__/ (root, 1 test) - observability/__tests__/ (1 test) - scanner/__tests__/ (1 test) - vector/__tests__/ (3 tests) - indexer/__tests__/ (3 tests) - indexer/utils/__tests__/ (3 tests) Issue: #52
Implement new configuration system for dev-agent with structured
schema, environment variable templating, and validation.
Implementation:
- New DevAgentConfig schema with version, repository, and mcp sections
- Environment variable templating support (${VAR_NAME} syntax)
- Config validation with detailed error messages
- Changed config file location to .dev-agent/config.json
- Backward compatibility with legacy fields
- Adapter configuration support for MCP adapters
Features:
- Structured repository configuration (path, excludePatterns, languages)
- MCP adapter configuration (enabled, source, settings)
- Environment variable resolution in config values
- Type-safe config loading and validation
- Config file discovery from current directory upward
Testing:
- Config validation covers all schema fields
- Environment variable resolution tested
- Backward compatibility maintained
Issue: #52
Update all CLI commands to use centralized storage paths instead
of project-local .dev-agent directories.
Implementation:
- Updated index, search, explore, plan, stats, update, clean, compact commands
- All commands now use getStoragePath() and getStorageFilePaths()
- Commands use centralized ~/.dev-agent/indexes/{repo-hash}/ structure
- Updated metadata handling to use centralized storage
- CLI registration updated for new storage command
Changes:
- index: uses centralized storage paths
- search: uses centralized storage paths
- explore: uses centralized storage paths
- plan: uses centralized storage paths
- stats: displays centralized storage information
- update: uses centralized storage paths
- clean: removes centralized storage instead of local files
- compact: uses centralized storage paths
Testing:
- All commands updated and verified
- Storage path resolution working correctly
Issue: #52
Update MCP server to use centralized storage paths for repository
indexes and GitHub state.
Implementation:
- Updated dev-agent-mcp.ts to use getStoragePath() and getStorageFilePaths()
- MCP server now uses centralized ~/.dev-agent/indexes/{repo-hash}/ structure
- Added infrastructure for lazy loading (prefixed with _ for future use)
- Updated RepositoryIndexer and GitHubIndexer to use centralized paths
Features:
- Centralized storage for all indexed repositories
- Lazy loading infrastructure (foundation for future optimization)
- Consistent storage paths across CLI and MCP server
Testing:
- MCP server updated and verified
- Storage path resolution working correctly
Issue: #52
prosdev
force-pushed
the
issue/storage-config-system
branch
from
f76ffa5 to
48d7dce
Compare
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.
Overview
Implements the foundation for centralized storage and configuration system as part of Issue #52.
Changes
Storage Infrastructure
~/.dev-agent/indexes/{repo-hash}/getStoragePath,getStorageFilePaths, etc.)metadata.jsonper repository)Configuration System
${VAR_NAME}syntax).dev-agent/config.jsonCLI Updates
dev storage migratecommand for migrating local indexesdev storage infocommand for displaying storage informationMCP Server Updates
_for future use)Test Organization
__tests__convention (15 test files moved)Helper Scripts
gh-issue-add-subs.sh) for managing issue relationshipsTesting
Related Issues
Closes #52
Commits
refactor(core): reorganize all tests to use __tests__ conventionfeat(cli): implement new config system with schema validationfeat(cli): update commands to use centralized storagefeat(mcp-server): update to use centralized storage