Repository navigation
skills(ankra-applications): document .ankra/ankra.yaml and its components block (ankra-8k4lo.14) - #430
Merged
Conversation
…ents block The setup PR carries the application descriptor; say what it holds (metadata, the deploy options contract) and that a components: block declares a monorepo's components, outranks every inferred signal, is the only way to remove a component, and is refused whole as the read_declared_components task. reference.md gets the Cadence-shaped declaration example. Both link the new docs reference page. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Myothas
marked this pull request as ready for review
October 5, 2026 19:04
Ankra AI reviewVerdict: looks good. Docs-only change adding documentation for the .ankra/ankra.yaml application descriptor and its components block. The content is consistent between SKILL.md and reference.md and describes sensible precedence semantics. One minor uncertainty about how the fenced code blocks are written in the markdown source. Reviewed commit 3e7c0e5. This review is read-only and advisory. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Narrow update to the embedded
ankra-applicationsskill (the embedded copy is canonical;ankraio/ankra-skillsdoes not exist).SKILL.md§2 (Review what Ankra generated): the setup PR also carries.ankra/ankra.yaml; whatmetadataandoptionsare, and that an optionalcomponents:block declares a monorepo's components, outranks the existing workflows, the AI proposal and structural detection, is the only way to remove a component, and is refused whole as theread_declared_componentssetup task with the recorded components kept.reference.md(Monorepo components): the declaration example (Go API at the root built fromdeploy/docker/Dockerfileplusweb/) and the name/path rules.Both link the new reference page https://docs.ankra.ai/reference/application-descriptor (ankraio/ankra-docs#478, merged). Schema source: cluster
enginekit/appdescriptor/components.go(cluster#3916).go test ./internal/skills/passes; the pre-commit gate (go test ./..., golangci-lint) passed.Bead: ankra-8k4lo.14
🤖 Generated with Claude Code