docs(gitops-api): onboarding repo-discovery design (F8)#213
Conversation
…ped F1/F7 Add the F8 design — repo-wide GitTarget discovery in the manifest-analyzer CLI/library, never the operator — and move the shipped F1/F7 feature docs into finished/, updating inbound links and the feature ladder. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
📝 WalkthroughWalkthroughDocumentation updates move several shipped GitOps API design docs to a new ChangesDocumentation Updates
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@docs/design/gitops-api/f8-repo-discovery-and-onboarding-scan.md`:
- Around line 120-125: Update the onboarding taxonomy so `kustomize-single`
matches the supported single-context Kustomize surface by including `bases`
alongside `namespace`, `resources`, `images`, and `replicas`. Adjust the table
in the design doc to reflect the same accepted inputs described by
`docs/configuration.md` and the README, ensuring any folder using local bases is
still recognized as valid by the `acceptedByOperator` logic.
- Around line 184-186: The prose in this design document contains a stray diff
marker before “CI gate”; remove the leading plus sign so the sentence reads as
normal markdown. Update the affected text near the “library function” discussion
to keep the wording clean and free of accidental patch syntax.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro Plus
Run ID: a24b0819-f062-41fd-8137-4778c4657d34
📒 Files selected for processing (9)
docs/architecture.mddocs/configuration.mddocs/design/gitops-api/README.mddocs/design/gitops-api/f8-repo-discovery-and-onboarding-scan.mddocs/design/gitops-api/finished/f1-images-replicas-edit-through.mddocs/design/gitops-api/finished/f7-higher-level-krm-documents.mddocs/design/gitops-api/kustomize-support-boundary-and-product-model.mddocs/design/gitops-api/unreflectable-edits-and-write-gating.mdinternal/manifestanalyzer/testdata/contextual-namespace/README.md
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Adds the F8 design — repo-wide
GitTargetdiscovery in themanifest-analyzerCLI/library (never the operator) — and files the shipped F1/F7 docs underfinished/, with inbound links and the feature ladder updated.Docs-only; the first implementation lands in a follow-up PR.
🤖 Generated with Claude Code
Summary by CodeRabbit
New Features
Documentation