Skip to content

Improve diagnostic contracts and prepare Mechanic v1.4.3 - #4

Merged
Falkicon merged 3 commits into
mainfrom
falk/review-repo-quality-issues
Sep 6, 2026
Merged

Falkicon merged 3 commits into
mainfrom
falk/review-repo-quality-issues

Conversation

@Falkicon

@Falkicon Falkicon commented Sep 6, 2026 •

Copy link
Copy Markdown
Owner

Changes

Diagnostics now use a shared client/account/character/profile target from queue creation through SavedVariables reads. The dashboard generates forms from registered command schemas, and both MCP adapters expose typed inputs and audited mutation metadata.

Release and sync operations provide previews, preflight checks, and partial-failure recovery guidance. Bounded command/parser metrics and passive addon snapshots make overhead observable. The review also fixes parser performance, service lifecycle, local request validation, queue serialization, and packaged dashboard assets.

Repository documentation and generated command examples now match the current implementation. This PR prepares bootstrap version 1.4.3 and main-addon version 1.3.4, with a neutral release queue. CurseForge packaging now places both addon folders at the archive root, directs externals to their loaded library paths, and excludes development files.

Validation

  • 243 Python tests passed locally with MCP and Lua 5.1 enabled.
  • Dashboard schema/lifecycle and Lua queue/lifecycle/overhead regressions passed.
  • Focused Ruff checks, formatting, and local Lua lint passed.
  • Installed-wheel smoke checks passed for both dashboard assets.
  • 49 local documentation links and 97 CLI invocation templates verified.
  • Packager folder-move logic tested on an isolated tracked-file export; both addon TOCs resolve at their install roots. 347 manifest references and both external-only library entrypoints were checked. Full remote external-library checkout was not performed.
  • All 11 GitHub CI jobs passed on final head abc6c7a, including the Windows/Linux Python matrix.

Live WoW behavior has not been verified after these changes. The release tag will be pushed on the merged main commit after CI and packaging checks.

@Falkicon
Falkicon merged commit f4828b8 into main Sep 6, 2026
11 checks passed
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.

1 participant