Add bounded AL code generation skill contracts#103
Conversation
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: bd6344b4-eafd-4a58-a1c9-5a2d96fa2938
Keep the existing workflow unchanged while making schema validation self-contained when jsonschema is unavailable. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: bd6344b4-eafd-4a58-a1c9-5a2d96fa2938
|
👋 Heads up @JesperSchulz — and cc maintainers — this PR introduces new top-level entries that aren't part of BCQuality's known repository structure:
This isn't a block — just a flag. 🚩 New top-level folders and files are usually unintended (a stray export, a tool's scratch dir, or content that meant to land inside an existing layer like If this was intentional and the new entry genuinely belongs at the repo root, a maintainer can review and merge as normal — no action needed beyond a quick sanity check. If it wasn't, please move the content into the right existing layer (or drop it) and push an update. 🙏 A maintainer will take a look before merging. |
Summary
al-code-generationleaf and a separate generation-only plugin bridgerequirement-specv1 andgenerated-files-reportv1 JSON Schemas with parseable examplesBackward compatibility
Legacy review contexts remain unchanged: callers without
actionoraccepted-outputscontinue to negotiatefindings-reportv1. Generation requires explicitaction: generate,requirement-spec, and acceptance ofgenerated-files-reportv1. Mixed review/generation inputs without action fail as ambiguous. Existing review bridge behavior is unchanged, and current BCAppsBCQuality remains generation-ineligible because it does not advertise the new capabilities.Consumer dependency and rollout
This PR produces contracts and create-only reports; it does not materialize files. Consumers must implement strict atomic runtime validation plus independent symlink, destination, byte-size, and ID-range enforcement before staging. Follow-up consumer stages are strict parser fixtures, a dry-run patch artifact, isolated compile/analyze/test validation, and an environment-approved draft PR. Delivery, approval, and publishing remain consumer-owned.
Validation