Jesper Schulz-Wedde
e0ebdd35c7
Add lifecycle error and privacy knowledge ( #99 )
...
* Add lifecycle error and privacy knowledge
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 95c06ad8-377d-4faa-8d07-06300b1c81ec
* Fix lifecycle privacy review findings
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: a26cd6d6-ff49-433e-bc53-f645c455ebdd
* Refine lifecycle privacy retrieval
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 95c06ad8-377d-4faa-8d07-06300b1c81ec
* Make review gates explicit
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 95c06ad8-377d-4faa-8d07-06300b1c81ec
---------
Co-authored-by: Jesper Schulz-Wedde <jesper.schulzwedde@microsoft.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
2026-07-14 12:52:56 +02:00
Jesper Schulz-Wedde
5706959e4a
Fix lifecycle compatibility guidance ( #93 )
...
* Fix lifecycle compatibility guidance
Correct high-confidence Business Central guidance and samples for upgrade tags, collectible errors, trigger semantics, obsoletion, events, interfaces, API contracts, and test transactions.
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: e05a43e7-6448-4d67-9c73-798523f5d945
* Address guidance review findings
Gate SecretText guidance to BC23 and clarify that the collectible-error sample intentionally emits a message-only blocking aggregate.
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: e05a43e7-6448-4d67-9c73-798523f5d945
---------
Co-authored-by: Jesper Schulz-Wedde <jesper.schulzwedde@microsoft.com>
2026-07-14 11:26:16 +02:00
Copilot
65f6f22157
Add open-ended bc-version range shorthand [N..]
...
A closed range like [23..28] wrongly implies guidance stops applying after
BC28, so a reviewer targeting BC29+ would not match the file. Introduce an
open-ended shorthand [N..] meaning ''version N and every later version''.
- validate_frontmatter.py: RANGE_SHORTHAND allows an optional upper bound;
expand_bc_version returns the normalized string ''N..'' for open-ended.
- read.md: document the fourth bc-version form and its matching rule
(matches target >= N; not enumerable).
- write.md: prefer [N..] over a closed range for a feature introduced in N
and not expected to be removed.
- Apply [23..] to the actionable-errors article (actionable errors shipped
in BC23 and are not version-bounded above).
- README: mention [N..] in the frontmatter example.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
2026-06-23 11:20:11 +02:00
Copilot
8901b83e07
Add error-handling knowledge domain and review leaf skill
...
Seed the first new AL knowledge domain toward issue #34 : a fully wired
error-handling domain that the review pipeline actually consumes.
- 3 knowledge articles (+ good/bad AL samples):
- prefer-errorinfo-for-actionable-errors (ErrorInfo Fix-it/Show-it actions)
- collect-validation-errors-with-errorbehavior (ErrorBehavior::Collect)
- errortype-internal-vs-client-for-diagnostics (ErrorType Internal vs Client)
- New leaf skill al-error-handling-review.md, modeled on al-performance-review
- Wire the leaf into al-code-review (sub-skills, Source, description)
- README: six -> seven leaf skills
Validated: frontmatter validator clean; knowledge-index check passes
(deterministic, full coverage, selection inputs intact).
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
2026-06-23 10:35:24 +02:00
