Skip to content

replace prediction with semantic intent - #140

Open
atkrad wants to merge 1 commit into
mainfrom
add-semantic-intent
Open

atkrad wants to merge 1 commit into
mainfrom
add-semantic-intent

Conversation

@atkrad

@atkrad atkrad commented Sep 16, 2026

Copy link
Copy Markdown
Member

Summary

  • Build the semantic plan from Previous, Live, and Desired intent. Cluster reads are GET and mapping only. Helm action comes from raw chart intent, not live drift.
  • Plan extras CRDs and the install target Namespace as intent. Skip OriginNamespace when the chart already has that Namespace. Keep a plan when an upgrade is missing its Namespace.
  • Show modified and missing drift on the semantic plan. Rewrite deployah plan --drift as GET plus Desired declared-surface projection minus spec-edit paths.

Default deployah plan and deployah deploy stay on the legacy planner.

Test plan

  • Unit tests added/updated
  • Scenario under scenarios/ (if behavior changes)
  • nix run .#lint / pre-commit clean
  • Manual smoke (command + expected result), if user-facing

Kind e2e TestE2E/TestSemanticPlanPrerequisites covers the non-mutating GET path.

Labels

  • One of: kind/feature, kind/bug, kind/docs, kind/chore
  • Add breaking-change if this breaks existing CLI or config behavior
  • Add skip-changelog for internal-only PRs that should not appear in notes

Checklist

  • Title is short and imperative (matches commit style)
  • Docs / CLI help updated when user-facing (README.md, docs/cli/)
  • No secrets or local-only paths in the diff

- compare Previous, Live, and Desired instead of SSA prediction
- derive Helm action from raw chart intent, not live drift
- rewrite legacy --drift as GET plus declared-surface projection
@atkrad atkrad added kind/feature New capability skip-changelog Exclude this PR from release notes labels Sep 16, 2026
@github-actions github-actions Bot added area/cli Commands, flags, UX copy area/deploy Deploy and plan session flow labels Sep 16, 2026
@codecov

codecov Bot commented Sep 16, 2026

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/cli Commands, flags, UX copy area/deploy Deploy and plan session flow kind/feature New capability skip-changelog Exclude this PR from release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant