Skip to content

docs(lint-backlog): Batch 4 from the Mendix quality-source review, plus seven skill notes - #253

Closed
MendixMau wants to merge 1 commit into
docs/lint-backlogfrom
docs/lint-backlog-batch4
Closed

MendixMau wants to merge 1 commit into
docs/lint-backlogfrom
docs/lint-backlog-batch4

Conversation

@MendixMau

Copy link
Copy Markdown
Owner

What changed and why

Stacked on #243 (retarget to master once it merges).

A desk review of public Mendix quality sources (Best Practice Recommender, QSM/AQM docs, ATS docs, Menditect docs, docs.mendix.com) mapped about 700 practice rows against the shipped rules, the backlog, the in-flight UX and Batch 1 rules, and the skills. Result:

  • process/lint-backlog.md: new "Batch 4" section with 19 rows (13–31; status idea / blocked / skill), the sources and date, the caveat that lintability was read from the mxcli lint source and no rule has been probed on a real model yet, a suggested 2–3 rules-per-PR grouping, and review notes (stock CONV005 demands SNIPPET_ where Mendix's naming page says SNIP_; MPR001's prefix regex accepts any capitalised prefix and lists SCH_/SE_ where Mendix uses SCE_; stock ids are now MPR001–012).
  • process/quality-sources-review-2026-10-08.md: the review record (already-covered table, candidates, skill-only items, out-of-scope list, contradictions).
  • Seven non-lintable practices added to their skills, a few lines each: indexes (modularize-domain.md), scheduled-event flows (microflow-loop-antipatterns.md), Atlas 4 CSS variables and an accessibility pointer (design-artifacts.md), accessibility floor (ui-preflight-pages.md), workflow targeting defaults (learned-workflow-patterns.md), test isolation (testing-shape.md), rollback semantics (learned-microflow-patterns.md), and the indexes cell in mendix-best-practices-index.md.

Field evidence

Desk review 2026-10-08; no model probed. Every row is idea, blocked or skill, so nothing ships as a rule here.

Checklist

  • Size cap: 11 files, docs only
  • Test tier: T0
  • Instrument rules: n/a
  • CHANGELOG line under Unreleased
  • render-routing --check, leak guard, check-pr-discipline, check-portability clean

🤖 Generated with Claude Code

https://claude.ai/code/session_01VJgWP5vEoAsNsJYqCDMGNw


Generated by Claude Code

Adds 19 backlog rows (13-31) plus review notes to process/lint-backlog.md, the review record
process/quality-sources-review-2026-10-08.md, and seven skill-only additions (indexes, scheduled
events, Atlas 4 tokens, accessibility floor, workflow targeting, test isolation, rollback semantics).
Lintability read from mxcli source only; no rule probed on a model yet.
Checks: leak guard clean, render-routing --check exit 0, check-pr-discipline clean,
check-portability clean, check-docs-numbering (0 staged docs).

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01VJgWP5vEoAsNsJYqCDMGNw
MendixMau added a commit that referenced this pull request Oct 8, 2026
Squash of integration/2026-10-08, which merges:

- #225 perf(ui-loop): test-stack-up.sh recognises a live mxcli run --local
- #231 fix(sync): sync-project.sh no longer exits 1 silently before its bin/ refresh
- #242 fix(init): local by default, no Dev Container
- #243 docs(lint): process/lint-backlog.md, the lint rollout plan
- #247 new(lint): the lint gate keeps a ledger
- #249 learn(test): skills/e2e-locators-that-lie.md
- #250 learn(skills): unhappy-path-testing.md
- #251 fix(testing-shape): mxcli test --local shares the deploy directory (#230)
- #252 fix(obligations): LOOK staleness reads the newest VALID AT stamp (#155)
- #253 docs(lint): quality-source review, Batch 4 backlog rows and skill additions
- #254 fix(lint): CONV020 works out of the box, page navigation counts as feedback (#229, #245)
- #255 fix(e2e): journey-runner rung 2 runs for a negative-only span claim (#150)
- #256 fix(gates): Stage 6 Surface, source-sufficiency title, ${JOURNEY_DIR} manifest token (#233, #234, #235)

Verified on the branch: tests/run-tests.sh 33/0, tests/test-lint-delivery.sh ALL GREEN, tests/wave2 73/75 (two environmental), render-routing --check, check-scripts, check-portability, leak guard, check-no-private-citations, check-pr-discipline all clean; scaffold smoke run without crashes.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01VJgWP5vEoAsNsJYqCDMGNw
@MendixMau

Copy link
Copy Markdown
Owner Author

Merged into master via integration PR #258 (squash 9dd9be4), on the maintainer's approval. Closing this PR; its branch is part of that squash.


Generated by Claude Code

@MendixMau MendixMau closed this Oct 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants