Skip to content

refactor: separate planning from execution - #23

Merged
ScottishTapWater merged 3 commits into
feat/executor-orchestrationfrom
feat/executor-planning-boundary
Oct 28, 2026
Merged

ScottishTapWater merged 3 commits into
feat/executor-orchestrationfrom
feat/executor-planning-boundary

Conversation

@ScottishTapWater

@ScottishTapWater ScottishTapWater commented Oct 28, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Make the planner include direct dependency IDs and the unchanged dependency projects and versions needed to execute its plan. Each package version is recorded once.
  • Remove ExecutionInputResolver: execution maps the supplied plan into build and archive-verification inputs without rediscovering projects, rebuilding the graph or reading release tags.
  • Move the shared plan model to Public.Definition and extract repository HEAD reading into a shared internal service.
  • Record the build working directory in the plan and remove the obsolete solution-path argument from execution. Preserve nested global.json selection.
  • Configure six ArchonAnalysers rules preventing references between internal planning, execution and rollback namespaces in either direction.

Validation

  • Full solution: 124 tests passed, none skipped.
  • All six forbidden-reference probes failed with ARCHON004; temporary probe code was removed.
  • Formatting and diff checks passed. Independent production and test reviews found no material issues.

Stacked on #22. This changes the public plan namespace and adds required dependency and working-directory information to the plan model.

@ScottishTapWater
ScottishTapWater force-pushed the feat/executor-planning-boundary branch from 2173a2f to c7c4d72 Compare October 28, 2026 16:27
@ScottishTapWater
ScottishTapWater merged commit 0fa1237 into main Oct 28, 2026
7 of 8 checks passed
@ScottishTapWater
ScottishTapWater deleted the feat/executor-planning-boundary branch October 28, 2026 17:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant