ace-git-commit 0.23.6 → 0.26.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 98cfe129ae1e16f3bbcf324e62dde4b952976f5573bdde91db6c8ca6a73df865
-  data.tar.gz: 3b77fe0dce89927d1eb286060d7399a7aacc70c8738a3478be5cc1971e1103fb
+  metadata.gz: 2599fd8c9470d524bf85ca5eb8b50e612a62936dbda4a5d75b0b2576361d8c2f
+  data.tar.gz: b1b83adf72a08ff94c13fdeecb70be43574431255fab3a144d01fb3c92afd1cd
 SHA512:
-  metadata.gz: e9e0f3360cfe5b4412ac2cfaca652f84215709372277b4003209d12ea34d61f3bd33eeb2c5daa3511366aa1b1a3d3ba991074316f2f33d10947071756f106a07
-  data.tar.gz: d5b440d00e56e37aeaac68fdb22185fb972a69384fb0f04d4fe5d2fa484244f307e686f01131ca28b6253b199e95cba8ae1f627a66654f43d4477ebedb830590
+  metadata.gz: 61cf4c04784c908d0f5257db3021fe1f3aa34b24212b6b7af8c950190fd23aa5c6f64b841ce8aa300dc00d416821f0b4863cc21c9956c813cd94fa4a067ae4a8
+  data.tar.gz: 5853eef8a0bd648f524254e3081f997f4d54ecfd9270150e090476601c5c5a460c7eec520462becd0fec3dcd518121480d27545ec10de05f1384500a9eae9bc2

data/.ace-defaults/git/commit.yml

@@ -19,4 +19,9 @@ git:
         - api
         - ui
         - docs
-        - test
+        - test
+
+  split:
+    enabled: true
+    strategy: config-scope
+    description: Automatically create multiple commits when multiple config scopes are detected; use --no-split to force a single commit.

data/CHANGELOG.md

@@ -7,6 +7,91 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.26.3] - 2026-04-24
+
+### Changed
+- Clarified first-setup commit recovery across quick-start, package docs, and workflow guidance so users can inspect staged setup files, use the deterministic `--only-staged --no-split -m` path, and treat LLM-backed generation as optional after readiness checks.
+
+## [0.26.2] - 2026-04-23
+
+
+### Changed
+- Clarified staged-only and setup-failure commit workflows, including explicit path-scoped command contracts for single-package commits and recovery guidance.
+- Added commit contract coverage for staged-first and fallback paths so split/no-split behavior is deterministic before automatic path-scoped commit flow.
+
+
+## [0.26.1] - 2026-04-22
+
+### Technical
+- Tightened `TS-COMMIT-001` Goal 7/8 staging contracts to require explicit-path staging and prove `results/` artifacts are excluded from the cached staged set before `--only-staged` execution.
+
+## [0.26.0] - 2026-04-22
+
+### Changed
+- Improved setup-time LLM failure diagnostics to include failed role/model context, provider/setup remediation commands, and a deterministic `--only-staged --no-split -m` fallback commit path.
+
+### Technical
+- Extended `TS-COMMIT-001` with goal 8 setup-failure guidance coverage and added fast tests for role/model failure guidance and explicit CLI error routing.
+
+## [0.25.4] - 2026-04-16
+
+### Technical
+- Split the retained dry-run and path-scoped commit E2E capture into explicit `dry-run/` and `path-only/` artifact sets so verification requires both halves of the public workflow.
+
+## [0.25.3] - 2026-04-16
+
+### Technical
+- Relaxed `TS-COMMIT-001` auto-split verification to prove split behavior and path scoping without depending on an exact commit-count shape from the coordinated commit set.
+
+## [0.25.2] - 2026-04-16
+
+### Fixed
+- Strengthened `TS-COMMIT-001` delete/rename verification around explicit final-state artifacts so the retained E2E flow no longer depends on Git rename heuristics from `git show --stat`.
+
+## [0.25.1] - 2026-04-16
+
+### Fixed
+- Relaxed dry-run E2E verification to accept equivalent proof that dry-run preserved HEAD and did not silently commit staged changes before the subsequent path-scoped commit.
+
+## [0.25.0] - 2026-04-15
+
+### Changed
+- Rewrote `TS-COMMIT-001` split/no-split E2E goals around explicit public
+  setup and impact-first verification, and added `TC-007-only-staged-contract`
+  coverage for the staged-only user journey.
+- Expanded `docs/usage.md` and CLI help text with reproducible split/no-split
+  setup guidance and explicit `--only-staged` git-state expectations.
+
+## [0.24.2] - 2026-04-13
+
+### Technical
+- Tightened the package version contract test to require a full semantic-version match instead of accepting partial matches.
+## [0.24.1] - 2026-04-13
+
+### Changed
+- Completed the batch i05 migration follow-through for this package and aligned it with the restarted `fast` / `feat` / `e2e` verification model.
+
+### Technical
+- Included in the coordinated assignment-driven patch release for batch i05 package updates.
+
+
+## [0.24.0] - 2026-04-12
+
+### Changed
+- Migrated package tests to the restarted `fast` / `feat` / `e2e` model by moving deterministic suite files into `test/fast/` and keeping scenario workflows in `test/e2e/`.
+- Added `unit-coverage-reviewed` metadata for `TS-COMMIT-001` and aligned scenario sandbox setup to support `${ACE_E2E_SOURCE_ROOT:-$PROJECT_ROOT_PATH}`.
+- Updated package docs (`README`, `docs/usage.md`, `docs/getting-started.md`) to teach the restarted testing contract (`ace-test`, `ace-test ... feat`, `ace-test-e2e`, `ace-test ... all`).
+
+### Changed
+- **ace-git-commit v0.23.8**: Release workflow now loads git context for commit guidance and JSON status data from bundled workflow sections.
+
+
+## [0.23.7] - 2026-04-10
+
+### Fixed
+- Restored packaged default split configuration (`git.split.enabled: true`, `git.split.strategy: config-scope`) and documented `--no-split` as the explicit single-commit override.
+- Added packaged-default regression coverage in `test/default_config_test.rb` to enforce the split-default contract.
+
 ## [0.23.6] - 2026-03-31
 
 ### Changed

data/README.md

@@ -36,6 +36,17 @@
 
 **Preview and control commit behavior safely** - use `--dry-run` to preview, `--only-staged` for strict staging, or `-m` for explicit messages, with configuration cascade from [ace-support-config](../ace-support-config) for project and user overrides.
 
+For the retained public verification flows, see the reproducible examples in the [Usage Guide](docs/usage.md#reproducible-split-and-no-split-setup) and [`--only-staged` expected git state](docs/usage.md#--only-staged-expected-git-state).
+
+## Testing Contract
+
+`ace-git-commit` follows the restarted package test model:
+
+- `ace-test ace-git-commit` for deterministic fast-loop coverage (`test/fast/`)
+- `ace-test ace-git-commit feat` when deterministic feature/contract coverage is present (`test/feat/`)
+- `ace-test-e2e ace-git-commit` for retained workflow scenarios (`test/e2e/`)
+- `ace-test ace-git-commit all` for full package verification
+
 ## Configuration
 
 Override the default LLM model in `.ace/git/commit.yml` (see [defaults](.ace-defaults/git/commit.yml) for all options), or replace any prompt template in `.ace-handbook/prompts/` to match your commit style.

data/handbook/prompts/git-commit.system.md

@@ -140,7 +140,7 @@ When multiple files are changed:
 Describe the action, not the content:
 - The subject must describe what this commit DOES to the codebase
 - Do NOT summarize the content of changed files as if that content is the change itself
-- Example: deleting a task spec about "rename X to Y" → `spec(task-272): remove specflow-rename task` NOT `spec(taskflow-rename): rename ace-taskflow to ace-specflow`
+- Example: deleting a task spec about "rename X to Y" → `spec(task-272): remove specflow-rename task` NOT `spec(taskflow-rename): rename ace-task to ace-specflow`
 - Example: deleting a deprecated module → `refactor(auth): remove legacy OAuth handler` NOT `refactor(auth): OAuth 1.0 authentication flow`
 
 When all changes are deletions:

data/handbook/workflow-instructions/git/commit.wf.md

@@ -5,6 +5,19 @@ purpose: commit workflow instruction
 ace-docs:
   last-updated: 2026-03-15
   last-checked: 2026-03-21
+bundle:
+  embed_document_source: true
+  params:
+    output: cache
+    max_size: 81920
+    timeout: 30
+  sections:
+    current_repository_status:
+      title: Current Repository Status
+      commands:
+        - git status -sb
+        - git diff --stat
+        - ace-git status --format json
 ---
 
 # Commit Workflow
@@ -46,6 +59,7 @@ ace-git-commit automatically:
    - Specific files: `ace-git-commit file1 file2`
    - Only staged: `ace-git-commit --only-staged`
    - Dry run first: `ace-git-commit --dry-run -i "$intention"`
+   - First setup snapshot after provider-readiness issues: `ace-git-commit --only-staged --no-split -m "chore: set up ace tooling"`
 
 3. **Verify result**:
    ```bash
@@ -58,6 +72,7 @@ ace-git-commit automatically:
 - `-m, --message`: Use direct message (bypass LLM)
 - `--model MODEL`: Override LLM model (e.g., gflash)
 - `-s, --only-staged`: Commit only staged changes
+- `--no-split`: Force one commit across detected scopes
 - `-n, --dry-run`: Preview without committing
 - `-d, --debug`: Enable debug output
 
@@ -72,4 +87,4 @@ ace-git-commit automatically:
 **Changes Committed:** [Brief summary of what was committed]
 **Commit Message:** [The generated/used message]
 **Files Modified:** [Number of files and brief description]
-**Status:** ✓ Complete | ✗ Failed with [reason]
+**Status:** ✓ Complete | ✗ Failed with [reason]

data/lib/ace/git_commit/atoms/gitignore_checker.rb

@@ -103,7 +103,7 @@ module Ace
         def extract_pattern(output)
           return nil if output.nil? || output.strip.empty?
 
-          # Format: ".gitignore:3:.ace-taskflow/**/reviews/    .ace-taskflow/v.0.9.0/reviews/review-report-gpro.md"
+          # Format: ".gitignore:3:.ace-task/**/reviews/    .ace-task/v.0.9.0/reviews/review-report-gpro.md"
           # We want the third field (the pattern)
           parts = output.strip.split("\t")
           if parts.length >= 2

data/lib/ace/git_commit/cli/commands/commit.rb

@@ -20,6 +20,9 @@ module Ace
 
             When no files are specified, all changes are staged.
             When files are provided, only those files are staged and committed.
+            When changes span configured scopes, commits may auto-split by scope.
+            Use --no-split to force one commit, or --only-staged to commit only
+            files already in the index.
 
             Configuration:
               Global config:  ~/.ace/git/commit.yml
@@ -33,7 +36,7 @@ module Ace
             "-i 'fix bug'                # With intention",
             "-m 'feat: add'              # With explicit message",
             "--only-staged                # Only staged changes",
-            "--no-split                   # Force a single commit"
+            "--no-split pkg-a pkg-b       # Force a single commit across scopes"
           ]
 
           # Define files as variadic argument (can be 0 or more)
@@ -43,7 +46,7 @@ module Ace
           option :intention, type: :string, aliases: %w[-i], desc: "Provide context for LLM message generation"
           option :message, type: :string, aliases: %w[-m], desc: "Use provided message directly (no LLM)"
           option :model, type: :string, desc: "Override default LLM model (e.g., glite, gflash)"
-          option :only_staged, type: :boolean, aliases: %w[-s], desc: "Commit only currently staged changes"
+          option :only_staged, type: :boolean, aliases: %w[-s], desc: "Commit only currently staged changes and preserve unstaged edits"
           option :staged, type: :boolean, desc: "Alias for --only-staged"
           option :dry_run, type: :boolean, aliases: %w[-n], desc: "Show what would be committed without doing it"
           option :force, type: :boolean, aliases: %w[-f], desc: "Force operation (for future use)"
@@ -80,6 +83,8 @@ module Ace
             raise Ace::Support::Cli::Error.new("Commit failed") unless success
           rescue GitError => e
             raise Ace::Support::Cli::Error.new(e.message)
+          rescue Ace::GitCommit::Error => e
+            raise Ace::Support::Cli::Error.new(e.message)
           rescue Interrupt
             raise Ace::Support::Cli::Error.new("Commit cancelled", exit_code: 130)
           end

data/lib/ace/git_commit/molecules/message_generator.rb

@@ -29,10 +29,11 @@ module Ace
         def generate(diff, intention: nil, files: [], config: nil)
           system_prompt = load_system_prompt
           user_prompt = build_user_prompt(diff, intention, files)
+          model = resolve_model(config)
 
           # Use QueryInterface with named parameters matching CLI
           response = Ace::LLM::QueryInterface.query(
-            resolve_model(config),
+            model,
             user_prompt,
             system: system_prompt,
             temperature: 0.7,
@@ -42,7 +43,7 @@ module Ace
 
           clean_commit_message(response[:text])
         rescue Ace::LLM::Error => e
-          raise Error, "Failed to generate commit message: #{e.message}"
+          raise Error, llm_failure_message("Failed to generate commit message", model, e)
         end
 
         # Generate commit messages for multiple groups in one LLM call
@@ -60,9 +61,10 @@ module Ace
 
           system_prompt = load_batch_system_prompt
           user_prompt = build_batch_user_prompt(groups_context, intention)
+          model = resolve_model(config)
 
           response = Ace::LLM::QueryInterface.query(
-            resolve_model(config),
+            model,
             user_prompt,
             system: system_prompt,
             temperature: 0.7,
@@ -77,7 +79,7 @@ module Ace
 
           raise Error, "Failed to generate batch commit messages: #{e.message}"
         rescue Ace::LLM::Error => e
-          raise Error, "Failed to generate batch commit messages: #{e.message}"
+          raise Error, llm_failure_message("Failed to generate batch commit messages", model, e)
         end
 
         private
@@ -260,10 +262,11 @@ module Ace
 
         def retry_batch_parse(groups_context, intention, previous_response, reason, config)
           warn "[ace-git-commit] Batch parse failed, retrying with strict JSON repair: #{reason}"
+          model = resolve_model(config)
 
           repair_prompt = build_batch_repair_user_prompt(groups_context, intention, previous_response, reason)
           repair_response = Ace::LLM::QueryInterface.query(
-            resolve_model(config),
+            model,
             repair_prompt,
             system: load_batch_system_prompt,
             temperature: 0.2,
@@ -275,6 +278,8 @@ module Ace
         rescue BatchParseError => e
           warn "[ace-git-commit] Batch parse retry failed: #{e.message}"
           nil
+        rescue Ace::LLM::Error => e
+          raise Error, llm_failure_message("Failed to generate batch commit messages", model, e)
         end
 
         def build_batch_repair_user_prompt(groups_context, intention, bad_response, reason)
@@ -299,6 +304,25 @@ module Ace
           config_override.fetch("model", @model)
         end
 
+        def llm_failure_message(prefix, model, error)
+          effective_model = model.to_s.empty? ? DEFAULT_MODEL : model
+          surface = setup_surface_label(effective_model)
+          lines = []
+          lines << "#{prefix} with #{surface} '#{effective_model}': #{error.message}"
+          lines << ""
+          lines << "LLM setup checks:"
+          lines << "  ace-llm --list-providers"
+          lines << "  ace-config doctor"
+          lines << ""
+          lines << "Fallback commit command:"
+          lines << '  ace-git-commit --only-staged --no-split -m "chore: set up ace tooling"'
+          lines.join("\n")
+        end
+
+        def setup_surface_label(model)
+          model.start_with?("role:") ? "ACE role" : "LLM model"
+        end
+
         # Load system prompt from template
         # @return [String] System prompt content
         def load_system_prompt

data/lib/ace/git_commit/version.rb

@@ -2,6 +2,6 @@
 
 module Ace
   module GitCommit
-    VERSION = '0.23.6'
+    VERSION = '0.26.3'
   end
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: ace-git-commit
 version: !ruby/object:Gem::Version
-  version: 0.23.6
+  version: 0.26.3
 platform: ruby
 authors:
 - Michal Czyz
 bindir: exe
 cert_chain: []
-date: 2026-04-01 00:00:00.000000000 Z
+date: 2026-04-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ace-support-cli