-
Notifications
You must be signed in to change notification settings - Fork 3.3k
Update installation commands in README to use correct repository URL #901
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Closed
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
Update installation commands in README to use correct repository URL
- Add Fork Management section to CLAUDE.md with sync workflows - Create FORK_CUSTOMIZATIONS.md to track fork-specific changes - Add prepare-sync.sh automation script for safe upstream syncs - Update .gitignore to exclude AI configuration files These additions provide complete documentation for managing this fork of github/spec-kit while preserving custom modifications during upstream synchronization.
docs: Add comprehensive fork management guidelines
… implementation planning from ARCHON
- Created VALIDATION_REPORT.md to document test results and issues found. - Added run_all_silent_tests.sh to execute all silent integration test suites. - Implemented test_mcp_detection.sh to validate MCP detection functions. - Developed test_silent_integration.sh to ensure all scripts operate silently. - Created test_state_management.sh to verify project/document/task ID mappings. - Added test_task_parsing.sh to check task parsing logic with project setup. - Implemented test_gitignore.sh to confirm .gitignore configuration. - Developed test_common_integration.sh to validate integration with common.sh. - Created test_error_handling.sh to ensure graceful degradation on errors. - Added test-archon-integration.sh as a comprehensive validation test suite.
…l support, extended integration hooks, webhook functionality, multi-project dashboard, and AI-assisted conflict resolution.
…egration tests and test runner scripts - Created `run-archon-tests.sh` to automate the execution of PowerShell tests. - Added `test-archon-common.Tests.ps1` for unit tests on common Archon functions, covering silent operation, state management, and cross-platform compatibility. - Implemented `test-archon-integration.Tests.ps1` to validate Archon integration scripts, including JSON generation, request file creation, and error handling. - Ensured all tests validate silent operation and proper error handling for missing or invalid feature directories.
Archon-Spec Kit
- Changed repo_owner from 'github' to 'aloyxa1226' - This ensures `specify init` downloads from our fork with Archon integration - Reason: Fork has custom Archon MCP integration not in upstream - Upstream merge strategy: manual (keep ours for fork, theirs for upstream features)
…tation improvements This commit implements comprehensive Archon MCP integration enhancements inspired by archon-example-workflow analysis, with full cross-platform script support and improved documentation for brownfield development. ## Archon Workflow Enhancements ### Templates Enhanced - **templates/commands/implement.md**: - Added one-task-at-a-time enforcement (Step 9) - Added comprehensive post-implementation validation phase (Step 10) - Validates TDD adherence (tests before code per Article III) - Enhanced completion report with detailed metrics (Step 11) - **templates/commands/plan.md**: - Added brownfield analysis mode for existing codebases - Automatic detection for projects with >20 files - Pattern discovery: architecture, naming, testing, integration points - Documents constraints in research.md ### Repository Documentation - **CLAUDE.md**: - Enhanced Section 4 with explicit Archon workflow (A-E subsections) - Added bulk task creation, one-task-at-a-time enforcement - Added critical rules checklist for Archon workflow - **NEW**: Added "CRITICAL: Script Consistency Rule" section - Mandatory dual-script development policy (bash + PowerShell) - Enforcement checklist, common pitfalls, best practices ## Cross-Platform Script Support ### Silent Archon Integration (New Files) - **scripts/bash/archon-inject-agent-docs.sh**: - Silently injects Archon workflow docs into agent files during `specify init` - Detects Archon MCP availability (completely silent) - Supports all 13 agent types (Claude, Copilot, Gemini, etc.) - Inserts after "## Project Overview" section - **scripts/powershell/archon-inject-agent-docs.ps1**: - PowerShell equivalent for Windows/cross-platform support - Identical functionality to bash version - Called when user selects --script ps ### CLI Integration - **src/specify_cli/__init__.py** (lines 974-998): - Added conditional script execution (bash vs PowerShell) - Silent failure pattern (capture_output=True, timeout=5) - Zero user visibility (completely transparent) - Cross-platform support based on --script flag ## Documentation Improvements ### README.md - Added brownfield mode documentation in quick start (Step 4) - Added comprehensive brownfield explanation in detailed walkthrough (STEP 4) - Documents automatic detection (>20 files threshold) - Explains pattern discovery and research.md integration ### FORK_CUSTOMIZATIONS.md - Documented all file modifications with merge strategies - Added sync history entry for Archon workflow enhancements - Listed adopted vs rejected patterns with rationale - Documented script consistency rule addition - Documented README brownfield documentation ## Key Design Decisions ### Adopted from archon-example-workflow: - ✅ One task in "doing" at a time enforcement - ✅ Post-implementation validation pattern (modified for TDD compliance) - ✅ Brownfield codebase analysis (inline in plan.md) - ✅ Silent Archon workflow doc injection ### Rejected from archon-example-workflow: - ❌ validator agent (violates Test-First Imperative) - ❌ codebase-analyst agent (replaced with inline pattern) - ❌ /create-plan command (inferior to /speckit.plan) ## Cross-Platform Consistency All script modifications now enforce dual-script development: - ALWAYS update BOTH bash AND PowerShell variants - Test BOTH variants before committing - Update command templates to reference both script paths - Best practice: bash first, then immediate PowerShell port ## Impact - Enhanced SDD workflow with proven Archon best practices - Maintained constitutional integrity (Article III: Test-First Imperative) - Zero user visibility for Archon integration (silent operation) - Full cross-platform support (Unix/Linux/macOS/Windows) - Brownfield mode now discoverable in README - Script consistency policy prevents future cross-platform gaps 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
- Changed workflow trigger to support both main and fork-main branches - Modified get-next-version.sh to use fork-v* tag prefix - Prevents conflicts with upstream github/spec-kit tags (v0.0.69) - Fork releases will now use fork-v0.0.x versioning scheme Fixes #issue-with-release-not-creating Changes: - .github/workflows/release.yml: branches: [ main, fork-main ] - .github/workflows/scripts/get-next-version.sh: Use fork-v* pattern Next steps: 1. Push this commit to fork-main 2. Create initial tag: git tag fork-v0.0.1 && git push origin fork-v0.0.1 3. Run workflow to create fork-v0.0.2 release
- Updated version validation regex to accept both v0.0.0 and fork-v0.0.0 - Fixes release workflow failure at package creation step Error was: 'Version must look like v0.0.0' Now accepts: v0.0.0 or fork-v0.0.0
## Changes ### Archon Integration (Issue #1) - Added silent Archon sync hooks to ALL workflow commands - /speckit.specify: Push spec.md to Archon after creation - /speckit.plan: Pull before planning, push after (plan.md, research.md, data-model.md) - /speckit.tasks: Pull before tasks, push tasks.md + create Archon tasks - /speckit.implement: Already had sync (no changes) - Result: Documents now sync automatically to/from Archon MCP during workflow ### Interactive Brownfield Filtering (Issue #2) - Added auto-detection of large directories (>1000 files or >100MB) - Interactive confirmation with 4 options: * A: Exclude all recommended (default) * B: No filtering (analyze everything) * C: Custom selection (pick individually) * D: Show sample files first - Environment variable SPECKIT_IGNORE for automation - NO dotfiles created - everything done interactively - Exclusions documented in research.md with metrics ## Impact - Archon users: Documents sync automatically (completely silent) - Brownfield users: AI asks permission before excluding large dirs - No breaking changes: Additive enhancements with graceful degradation - Clean repos: No .speckit-ignore file clutter ## Modified Files - templates/commands/specify.md (added Archon push hook) - templates/commands/plan.md (added sync hooks + interactive filtering) - templates/commands/tasks.md (added sync hooks + task creation) - FORK_CUSTOMIZATIONS.md (documented changes) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
@localden - apologies for raising the PR. it was raise in error. |
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.
No description provided.