ace-task 0.41.0 → 0.41.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a67866ad3602fb4570aca58dc63b0f60ea7575ce6baae62a6c1d6414970651c9
-  data.tar.gz: 7e547e93da96598b520866b605cf0368ec5beb2ff90c313c7118e649f029cae8
+  metadata.gz: 6e932999b7042c67c443a0221b3460edad42e09711a1f69e018f6a44473065f6
+  data.tar.gz: f993abd4008d257ce10e8dc105633294c31a1753fa76e5ea35e8355be3618ee2
 SHA512:
-  metadata.gz: 9a86efa97d4a87a6e3de9561bb65a2cd9c0456ede8ece70e34c4a665776c4602c26977aa3557699c774118133571734c3266eba9b4523912225cf71f0f4aa4fd
-  data.tar.gz: e0301e6f51931b8b7320b01e362bab0d7988f67607660b0374e87b70b9bec524ed41e1640d7692bb914dc201553becb1a13057b036d949d6fef930ff0378929e
+  metadata.gz: 6d508f8f9975cc717d0b4b788e824c5e800b5f5f5aae9e36639da1f9d98c9d6b95010371f8091ee18016d7548a23ffd6a7002b194cb4146e96e9dc2b1f8b5094
+  data.tar.gz: '08af4e6b32c752ce7253a147e98cdbfb8c5d2a930f8b51e06dcb5798403d85a592dadde99d854946c009cdf4e7774fa3b4e2eadce303d7ebc190d71e39a5c19b'

data/CHANGELOG.md

@@ -7,6 +7,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.41.1] - 2026-11-10
+
+### Fixed
+
+- Serialize subtask allocation/reparenting across processes and path aliases. Resolve accessible task references without weakening full inventory errors; preserve private ignored-task ownership.
+
 ## [0.41.0] - 2026-10-17
 
 ### Added

data/README.md

@@ -21,7 +21,7 @@
 > `--git-commit` is refused for Git-ignored tasks, and completed ignored tasks
 > are neither auto-archived nor moved by `doctor --auto-fix`. A lab may prepare
 > `.ace-tasks/_overseer/` (gitignored) and manage it with
-> `ace-task --in _overseer`; the delegation tools are role- and folder-name
+> `ace-task list --in _overseer`; the delegation tools are role- and folder-name
 > agnostic and do not reject management tasks. The canonical flow is
 > [docs/lab-flow.md](../docs/lab-flow.md).
 

data/lib/ace/task/molecules/subtask_allocation_lock.rb

@@ -0,0 +1,23 @@
+# frozen_string_literal: true
+
+module Ace
+  module Task
+    module Molecules
+      # The parent directory inode is shared across processes and path aliases.
+      # Lock it across allocation and publication without adding task-store files.
+      module SubtaskAllocationLock
+        def self.synchronize(parent_path)
+          File.open(parent_path, File::RDONLY) do |directory|
+            directory.flock(File::LOCK_EX)
+            held = directory.stat
+            current = File.stat(parent_path)
+            unless current.directory? && [held.dev, held.ino] == [current.dev, current.ino]
+              raise ArgumentError, "Parent task moved while waiting for subtask allocation; retry"
+            end
+            yield
+          end
+        end
+      end
+    end
+  end
+end

data/lib/ace/task/molecules/subtask_creator.rb

@@ -4,6 +4,7 @@ require "fileutils"
 require_relative "../atoms/task_id_formatter"
 require_relative "../atoms/task_frontmatter_defaults"
 require_relative "task_loader"
+require_relative "subtask_allocation_lock"
 
 module Ace
   module Task
@@ -44,32 +45,34 @@ module Ace
         )
           raise ArgumentError, "Title is required" if title.nil? || title.strip.empty?
 
-          paths = destination_paths_for(parent_task, title)
-          subtask_id = paths[:id]
-          subtask_dir = paths[:dir]
-          FileUtils.mkdir_p(subtask_dir)
-
-          # Build frontmatter
-          effective_status = status || @config.dig("task", "default_status") || "pending"
-          frontmatter = Atoms::TaskFrontmatterDefaults.build(
-            id: subtask_id,
-            status: effective_status,
-            priority: priority,
-            tags: tags,
-            created_at: time,
-            parent: parent_task.id,
-            estimate: estimate,
-            remote_issue: remote_issue
-          )
-
-          # Write spec file
-          spec_file = paths[:spec]
-          content = build_spec_content(frontmatter: frontmatter, title: title)
-          File.write(spec_file, content)
-
-          # Load and return
-          loader = TaskLoader.new
-          loader.load(subtask_dir, id: subtask_id, load_subtasks: false)
+          SubtaskAllocationLock.synchronize(parent_task.path) do
+            paths = destination_paths_for(parent_task, title)
+            subtask_id = paths[:id]
+            subtask_dir = paths[:dir]
+            FileUtils.mkdir_p(subtask_dir)
+
+            # Build frontmatter
+            effective_status = status || @config.dig("task", "default_status") || "pending"
+            frontmatter = Atoms::TaskFrontmatterDefaults.build(
+              id: subtask_id,
+              status: effective_status,
+              priority: priority,
+              tags: tags,
+              created_at: time,
+              parent: parent_task.id,
+              estimate: estimate,
+              remote_issue: remote_issue
+            )
+
+            # Write spec file
+            spec_file = paths[:spec]
+            content = build_spec_content(frontmatter: frontmatter, title: title)
+            File.write(spec_file, content)
+
+            # Load and return
+            loader = TaskLoader.new
+            loader.load(subtask_dir, id: subtask_id, load_subtasks: false)
+          end
         end
 
         # Non-mutating preview of the directory and spec file a subtask create

data/lib/ace/task/molecules/task_reparenter.rb

@@ -29,7 +29,7 @@ module Ace
           when "none"
             promote_to_standalone(task)
           when "self"
-            convert_to_orchestrator(task)
+            SubtaskAllocationLock.synchronize(task.path) { convert_to_orchestrator(task) }
           else
             demote_to_subtask(task, parent_ref: target, resolve_ref: resolve_ref)
           end
@@ -110,6 +110,10 @@ module Ace
             raise ArgumentError, "Cannot convert subtask #{task.id} to orchestrator — promote first"
           end
 
+          unless scan_existing_subtask_chars(task.path, task.id).empty?
+            raise ArgumentError, "Task #{task.id} already has subtasks; cannot convert it again"
+          end
+
           paths = orchestrator_paths(task)
           subtask_id = paths[:id]
           subtask_dir = paths[:dir]
@@ -155,37 +159,39 @@ module Ace
             raise ArgumentError, "Cannot reparent task to itself"
           end
 
-          paths = demoted_paths(task, parent_task)
-          new_id = paths[:id]
-          new_dir = paths[:dir]
-          new_spec_basename = paths[:spec_basename]
+          SubtaskAllocationLock.synchronize(parent_task.path) do
+            paths = demoted_paths(task, parent_task)
+            new_id = paths[:id]
+            new_dir = paths[:dir]
+            new_spec_basename = paths[:spec_basename]
 
-          raise ArgumentError, "Destination already exists: #{new_dir}" if File.exist?(new_dir)
+            raise ArgumentError, "Destination already exists: #{new_dir}" if File.exist?(new_dir)
 
-          # Move folder into parent
-          FileUtils.mv(task.path, new_dir)
+            # Move folder into parent
+            FileUtils.mv(task.path, new_dir)
 
-          # Rename spec file
-          old_spec_basename = File.basename(task.file_path)
-          old_spec_in_new_dir = File.join(new_dir, old_spec_basename)
-          new_spec_path = File.join(new_dir, new_spec_basename)
+            # Rename spec file
+            old_spec_basename = File.basename(task.file_path)
+            old_spec_in_new_dir = File.join(new_dir, old_spec_basename)
+            new_spec_path = File.join(new_dir, new_spec_basename)
 
-          File.rename(old_spec_in_new_dir, new_spec_path) if old_spec_basename != new_spec_basename
+            File.rename(old_spec_in_new_dir, new_spec_path) if old_spec_basename != new_spec_basename
 
-          # Update frontmatter: new ID, set parent
-          update_frontmatter(new_spec_path) do |fm|
-            fm["id"] = new_id
-            fm["parent"] = parent_task.id
-          end
+            # Update frontmatter: new ID, set parent
+            update_frontmatter(new_spec_path) do |fm|
+              fm["id"] = new_id
+              fm["parent"] = parent_task.id
+            end
 
-          # Descendant identities hang off the parent ID: rewrite their
-          # frontmatter and spec filenames so persisted IDs stay authoritative.
-          id_map = {task.id => new_id}
-          rewrite_descendants(new_dir, old_parent_id: task.id, new_parent_id: new_id, id_map: id_map)
-          rewrite_dependency_references(id_map)
+            # Descendant identities hang off the parent ID: rewrite their
+            # frontmatter and spec filenames so persisted IDs stay authoritative.
+            id_map = {task.id => new_id}
+            rewrite_descendants(new_dir, old_parent_id: task.id, new_parent_id: new_id, id_map: id_map)
+            rewrite_dependency_references(id_map)
 
-          loader = TaskLoader.new
-          loader.load(new_dir, id: new_id)
+            loader = TaskLoader.new
+            loader.load(new_dir, id: new_id)
+          end
         end
 
         # Rewrite dependency references in every task spec after an ID map

data/lib/ace/task/molecules/task_scanner.rb

@@ -35,8 +35,8 @@ module Ace
 
         # Scan for all primary tasks (excludes subtask folders).
         # @return [Array<ScanResult>] Sorted scan results
-        def scan
-          @scanner.scan.reject { |sr| subtask_folder?(sr.folder_name) }
+        def scan(accessible_only: false)
+          @scanner.scan(accessible_only: accessible_only).reject { |sr| subtask_folder?(sr.folder_name) }
         end
 
         # Scan and filter by special folder or virtual filter.

data/lib/ace/task/organisms/task_manager.rb

@@ -877,7 +877,7 @@ module Ace
 
         def resolve_scan_result(ref)
           scanner = Molecules::TaskScanner.new(@root_dir)
-          scan_results = scanner.scan
+          scan_results = scanner.scan(accessible_only: true)
           resolver = Molecules::TaskResolver.new(scan_results)
           resolver.resolve(ref)
         end

data/lib/ace/task/version.rb

@@ -2,6 +2,6 @@
 
 module Ace
   module Task
-    VERSION = "0.41.0"
+    VERSION = "0.41.1"
   end
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: ace-task
 version: !ruby/object:Gem::Version
-  version: 0.41.0
+  version: 0.41.1
 platform: ruby
 authors:
 - Michal Czyz
 bindir: exe
 cert_chain: []
-date: 2026-10-20 00:00:00.000000000 Z
+date: 2026-11-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ace-support-core
@@ -44,6 +44,9 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0.15'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.15.17
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -51,6 +54,9 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0.15'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.15.17
 - !ruby/object:Gem::Dependency
   name: ace-support-markdown
   requirement: !ruby/object:Gem::Requirement
@@ -199,6 +205,7 @@ files:
 - lib/ace/task/molecules/issue_link.rb
 - lib/ace/task/molecules/issue_sync_adapter.rb
 - lib/ace/task/molecules/path_utils.rb
+- lib/ace/task/molecules/subtask_allocation_lock.rb
 - lib/ace/task/molecules/subtask_creator.rb
 - lib/ace/task/molecules/task_config_loader.rb
 - lib/ace/task/molecules/task_creator.rb