BC Observatoryunofficial, made by waldo
code change · featureAL Developmentofficial - Microsoftreviewed - checked by Opus

#157 18 more AL/BC patterns: data-modeling, testing, style, security, error-handling, ui, upgrade, web-services, appsource

BCQuality adds 18 new knowledge articles covering data modeling, testing, style, security, error handling, UI, upgrade, web services, and AppSource patterns with code examples and best practices.

Open in Claude157.mdPull request #157

Pull request · merged into main on 2026-09-29 by MichaelDieringer · 59 files (+1612 -2), 59 AL · feature · tier official · reviewed (checked by Opus)

What changed

  • 18 articles across data modeling (WorkDate, Media for pictures, table-type conventions), testing (BCPT, tagging, test structure) and style
  • Security, error handling, UI, upgrade, web services and AppSource guidance on permission sets, rollback-safe logging, page types, upgrade tag nesting, API write access and version bumps
  • Sibling good and bad AL samples are included where a code contrast helps
  • Four drafted articles were dropped as duplicates or overlaps of existing files
  • Review skills updated to reference the new articles

AL objects touched

  • microsoft/knowledge/appsource/release-must-update-app-version.md: added, not in the BC? snapshot yet
  • microsoft/knowledge/data-modeling/code-must-not-change-workdate.bad.al: added, not in the BC? snapshot yet
  • microsoft/knowledge/data-modeling/code-must-not-change-workdate.good.al: added, not in the BC? snapshot yet
  • microsoft/knowledge/data-modeling/code-must-not-change-workdate.md: added, not in the BC? snapshot yet
  • microsoft/knowledge/data-modeling/pictures-must-use-media-not-blob.bad.al: added, not in the BC? snapshot yet
  • microsoft/knowledge/data-modeling/pictures-must-use-media-not-blob.good.al: added, not in the BC? snapshot yet
  • microsoft/knowledge/data-modeling/pictures-must-use-media-not-blob.md: added, not in the BC? snapshot yet
  • microsoft/knowledge/data-modeling/table-design-must-match-bc-table-type-conventions.bad.al: added, not in the BC? snapshot yet
  • microsoft/knowledge/data-modeling/table-design-must-match-bc-table-type-conventions.good.al: added, not in the BC? snapshot yet
  • microsoft/knowledge/data-modeling/table-design-must-match-bc-table-type-conventions.md: added, not in the BC? snapshot yet
  • microsoft/knowledge/error-handling/defensive-vs-offensive-code-must-match-blast-radius.bad.al: added, not in the BC? snapshot yet
  • microsoft/knowledge/error-handling/defensive-vs-offensive-code-must-match-blast-radius.good.al: added, not in the BC? snapshot yet
  • microsoft/knowledge/error-handling/defensive-vs-offensive-code-must-match-blast-radius.md: added, not in the BC? snapshot yet
  • microsoft/knowledge/error-handling/log-writes-must-survive-rollback.bad.al: added, not in the BC? snapshot yet
  • microsoft/knowledge/error-handling/log-writes-must-survive-rollback.good.al: added, not in the BC? snapshot yet
  • microsoft/knowledge/error-handling/log-writes-must-survive-rollback.md: added, not in the BC? snapshot yet
  • microsoft/knowledge/security/exposed-objects-must-be-in-a-permission-set.bad.al: added, not in the BC? snapshot yet
  • microsoft/knowledge/security/exposed-objects-must-be-in-a-permission-set.good.al: added, not in the BC? snapshot yet
  • microsoft/knowledge/security/exposed-objects-must-be-in-a-permission-set.md: added, not in the BC? snapshot yet
  • microsoft/knowledge/style/al-comments-must-not-restate-what-code-already-shows.bad.al: added, not in the BC? snapshot yet
  • microsoft/knowledge/style/al-comments-must-not-restate-what-code-already-shows.good.al: added, not in the BC? snapshot yet
  • microsoft/knowledge/style/al-comments-must-not-restate-what-code-already-shows.md: added, not in the BC? snapshot yet
  • microsoft/knowledge/style/pages-must-not-contain-business-logic.bad.al: added, not in the BC? snapshot yet
  • microsoft/knowledge/style/pages-must-not-contain-business-logic.good.al: added, not in the BC? snapshot yet
  • microsoft/knowledge/style/pages-must-not-contain-business-logic.md: added, not in the BC? snapshot yet
  • microsoft/knowledge/style/source-organized-by-feature-not-object-type.md: added, not in the BC? snapshot yet
  • microsoft/knowledge/testing/bcpt-scenarios-must-be-app-specific.bad.al: added, not in the BC? snapshot yet
  • microsoft/knowledge/testing/bcpt-scenarios-must-be-app-specific.good.al: added, not in the BC? snapshot yet
  • microsoft/knowledge/testing/bcpt-scenarios-must-be-app-specific.md: added, not in the BC? snapshot yet
  • microsoft/knowledge/testing/given-blocks-must-cover-full-precondition-chain.bad.al: added, not in the BC? snapshot yet
  • microsoft/knowledge/testing/given-blocks-must-cover-full-precondition-chain.good.al: added, not in the BC? snapshot yet
  • microsoft/knowledge/testing/given-blocks-must-cover-full-precondition-chain.md: added, not in the BC? snapshot yet
  • microsoft/knowledge/testing/test-feature-scenario-tags.bad.al: added, not in the BC? snapshot yet
  • microsoft/knowledge/testing/test-feature-scenario-tags.good.al: added, not in the BC? snapshot yet
  • microsoft/knowledge/testing/test-feature-scenario-tags.md: added, not in the BC? snapshot yet
  • microsoft/knowledge/testing/test-one-when-per-test.bad.al: added, not in the BC? snapshot yet
  • microsoft/knowledge/testing/test-one-when-per-test.good.al: added, not in the BC? snapshot yet
  • microsoft/knowledge/testing/test-one-when-per-test.md: added, not in the BC? snapshot yet
  • microsoft/knowledge/testing/ui-test-codeunit-naming.bad.al: added, not in the BC? snapshot yet
  • microsoft/knowledge/testing/ui-test-codeunit-naming.good.al: added, not in the BC? snapshot yet
  • microsoft/knowledge/testing/ui-test-codeunit-naming.md: added, not in the BC? snapshot yet
  • microsoft/knowledge/ui/page-design-must-match-bc-page-type-conventions.bad.al: added, not in the BC? snapshot yet
  • microsoft/knowledge/ui/page-design-must-match-bc-page-type-conventions.good.al: added, not in the BC? snapshot yet
  • microsoft/knowledge/ui/page-design-must-match-bc-page-type-conventions.md: added, not in the BC? snapshot yet
  • microsoft/knowledge/upgrade/upgrade-tag-logic-must-not-nest-deeply.bad.al: added, not in the BC? snapshot yet
  • microsoft/knowledge/upgrade/upgrade-tag-logic-must-not-nest-deeply.good.al: added, not in the BC? snapshot yet
  • microsoft/knowledge/upgrade/upgrade-tag-logic-must-not-nest-deeply.md: added, not in the BC? snapshot yet
  • microsoft/knowledge/web-services/api-page-least-privilege-write-access.bad.al: added, not in the BC? snapshot yet
  • microsoft/knowledge/web-services/api-page-least-privilege-write-access.good.al: added, not in the BC? snapshot yet
  • microsoft/knowledge/web-services/api-page-least-privilege-write-access.md: added, not in the BC? snapshot yet
  • microsoft/skills/review/al-appsource-review.md: modified, not in the BC? snapshot
  • microsoft/skills/review/al-data-modeling-review.md: modified, not in the BC? snapshot
  • microsoft/skills/review/al-error-handling-review.md: modified, not in the BC? snapshot
  • microsoft/skills/review/al-security-review.md: modified, not in the BC? snapshot
  • microsoft/skills/review/al-style-review.md: modified, not in the BC? snapshot
  • microsoft/skills/review/al-testing-review.md: modified, not in the BC? snapshot
  • microsoft/skills/review/al-ui-review.md: modified, not in the BC? snapshot
  • microsoft/skills/review/al-upgrade-review.md: modified, not in the BC? snapshot
  • microsoft/skills/review/al-web-services-review.md: modified, not in the BC? snapshot

Context

  • Systems: development
  • From the description: “18 more knowledge articles generalized from patterns observed across real AppSource/PTE Business Central development”

Source: metadata of a merged pull request in microsoft/BCQuality (bcquality): title, labels, changed paths and counts, with a summary in our words. No source text and no description beyond one short quote (D10, D61).