fix: ddx init now creates complete .ddx directory structure#21
Open
mehdipiraee wants to merge 3 commits intoeasel:mainfrom
Open
fix: ddx init now creates complete .ddx directory structure#21mehdipiraee wants to merge 3 commits intoeasel:mainfrom
mehdipiraee wants to merge 3 commits intoeasel:mainfrom
Conversation
The installation URLs were pointing to ddx-tools/ddx which returns 404. Updated all references to use the correct repository easel/ddx. Files updated: - README.md: Fixed installation commands and repository references - docs/SETUP_GUIDE.md: Fixed installation URLs and GitHub links This resolves the 404 error users encounter when trying to install DDx. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Found and fixed all remaining incorrect repository references throughout: Documentation files (7): - docs/helix/02-design/architecture.md - docs/helix/02-design/contracts/CLI-001-core-commands.md - docs/helix/02-design/contracts/CLI-002-sync-commands.md - docs/helix/02-design/contracts/CLI-003-utility-commands.md - docs/helix/02-design/data-design.md - docs/helix/02-design/deployment.md - docs/helix/02-design/solution-designs/SD-002-upstream-synchronization.md Go source files (6): - cli/cmd/contribute.go - cli/cmd/e2e_test.go - cli/cmd/sync_acceptance_test.go - cli/cmd/sync_contract_test.go - cli/cmd/test_harness.go - cli/cmd/uninstall.go This completes the comprehensive fix for all 404-causing URL references throughout the entire codebase. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Resolves issue where ddx init only created .ddx.yml but not the .ddx/ directory, causing isInitialized() to return false and all subsequent commands to fail with "not initialized" errors. Changes: - ddx init now creates .ddx/ directory with required subdirectories: - .ddx/prompts/ (AI prompts and instructions) - .ddx/templates/ (Project and file templates) - .ddx/patterns/ (Reusable code patterns) - .ddx/configs/ (Tool configurations) This matches the expected structure documented in README Local-First Architecture. Testing: - Verified ddx init creates complete directory structure - Verified ddx diagnose now shows "Initialization: ✓ Initialized" - Verified ddx update no longer shows "not initialized" error - All commands that use isInitialized() now work correctly Fixes easel#20 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
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
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
Fixes the initialization bug where
ddx initonly created.ddx.ymlbut not the.ddx/directory, causing all subsequent DDx commands to fail with "not initialized" errors.Problem
After running
ddx init, users would see:ddx diagnose→ "Initialization: ✗ Not initialized"ddx update→ "❌ Not in a DDx project. Run 'ddx init' first."ddx contribute→ "❌ Not in a DDx project. Run 'ddx init' first."This happened because
isInitialized()checks for the.ddx/directory:But
ddx initonly created.ddx.yml, not.ddx/directory.Solution
Modified
ddx initto create the complete directory structure as documented in README:Changes
.ddx/:prompts/- AI prompts and instructionstemplates/- Project and file templatespatterns/- Reusable code patternsconfigs/- Tool configurationsTesting
Before Fix:
ddx init ddx diagnose # Showed "Initialization: ✗ Not initialized"After Fix:
Verification
ddx initcreates complete directory structureddx diagnoseshows "Initialization: ✓ Initialized"ddx updaterecognizes project as initializedddx contributerecognizes project as initializedFiles Changed
cli/cmd/init.go- Added directory structure creationResolves #20
🤖 Generated with Claude Code