Authoring pilot: al-api-page-author skill + code-artifact output kind#61
Open
JesperSchulz wants to merge 1 commit into
Open
Authoring pilot: al-api-page-author skill + code-artifact output kind#61JesperSchulz wants to merge 1 commit into
JesperSchulz wants to merge 1 commit into
Conversation
Add the first authoring action skill (al-api-page-author) and extend the stable DO contract with a second output kind (code-artifact) plus the object-spec input. Reuses existing web-services API knowledge with no new knowledge articles; article count unchanged. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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
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.
Summary
This is the first authoring action skill for BCQuality. Every action skill today is a review skill that consumes a diff and emits a
findings-report; this PR adds the authoring half — a skill that consumes an abstract spec and generates Business Central AL code.The pilot is an API-page author: given an
object-spec(target table, entity naming, fields to expose, read-only flag), it generates a correctPageType = APIpage applying the existing curated web-services API guidance.What's in this PR
microsoft/skills/author/al-api-page-author.md(newauthor/family) — a leaf authoring skill mirroring the canonicalal-web-services-reviewSource → Relevance → Worklist → Action → Output shape, but emitting generated AL. It applies the existing web-services knowledge (required API properties, SystemId-as-key, read-only lockdown, committed-data reads, bound actions, additive versioning) and emits unresolved spec ambiguities asopen-questions/ per-artifactnoteswith clearly-labeled placeholders rather than invented values.skills/do.md(findings-reportschema is untouched:object-specto the standardinputsand defines it (abstract description of the BC object(s) to generate, supplied by the orchestrator for authoring tasks).outputsprose to state two output kinds are defined —findings-report(review) andcode-artifact(authoring) — keeping the single-element list.### Code-artifact output (authoring skills)subsection: a JSON schema that reuses the findings-report envelope (skill,outcomeenum,outcome-reason, JSON-validity/escaping rules,suppressed,sub-results,skipped-sub-skills) and swapsfindings/countsforartifacts/open-questions, with field semantics forartifacts[].content(full AL as one escaped JSON string),path,references(empty ⇒ confidence capped at medium, mirroring the additive agent-findings principle),notes, andopen-questions. Documents that acode-artifactmaps to file creation/scaffolding rather than PR comments..github/scripts/validate_frontmatter.py— allow-list updates only, no rule-logic changes:object-specadded toSTANDARD_INPUTS(R17 no longer warns) andcode-artifactadded toALLOWED_OUTPUTS(R18 no longer errors).README.md/agent-consumption.md— minimal docs: note that the action-skill family also includes anauthorgroup that generates code (emittingcode-artifact), withal-api-page-author.mdas the first example; mermaid diagram unchanged.No duplicated knowledge
This pilot reuses the existing
microsoft/knowledge/web-services/*articles — no new knowledge articles were added (curating duplicates would fail the repo's remedial-admission test). Article count is unchanged.Validation
python .github/scripts/validate_frontmatter.py --root .→ 0 errors / 0 warningspwsh .github/scripts/Test-KnowledgeIndex.ps1 -Root .→ PASSED, 192 articles, deterministic, full coverage (unchanged)pwsh ./tools/Build-KnowledgeIndex.ps1→ builds, 192 articlesauthor/, is not anal-*-review.mdleaf, and is not registered inal-code-review.mdsub-skills.Do not merge — opening for review.