Skip to content

docs: clarify lintdiff publication command safety - #5640

Merged
Yuchao Yan (msyyc) merged 1 commit into
feature/lintdiff-migration-newfrom
docs/lintdiff-publication-command-safety
Oct 10, 2026
Merged

Yuchao Yan (msyyc) merged 1 commit into
feature/lintdiff-migration-newfrom
docs/lintdiff-publication-command-safety

Conversation

@msyyc

Copy link
Copy Markdown
Member

Summary

Clarify Windows publication command safety for the LintDiff queue and review loop.

  • Separate read-only identity checks from short rooted staging, commit, push, and publication steps.
  • Validate multi-line PowerShell publication syntax before executing it.
  • Use native Git exit codes, not stdout truthiness, for outputless commands.
  • Inspect the actual refreshed diagnostic-snapshot set instead of a guessed file count.

Evidence

The LroExtension workflow in #5639 completed its required review-fix validation but stopped before staging because a large PowerShell publication wrapper had a missing closing parenthesis. Earlier outputless Git wrappers and a guessed five-snapshot guard also obstructed valid work; the actual nine changed snapshots were independently verified as diagnostic-message replacements.

The primary review loop is stopped. This independent PR changes only skill documentation and does not resume that workflow, alter rule code, or grant an additional correction attempt. Parser rejection still counts as a command failure under the existing finite recovery policy, despite zero native mutation attempts.

Validation

Formatted only the two changed Markdown files with the existing Prettier 3.9.5 installation and identical repository configuration. Checked the complete diff, relative documentation links, staged scope, and git diff --check.

No dependency installation, submodule initialization, package build, corpus rerun, CI retry, or additional Copilot review loop is required for this documentation-only PR.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 625c0cd9-bacc-4f9c-8b96-4ded1932a30a
@msyyc
Yuchao Yan (msyyc) marked this pull request as ready for review October 10, 2026 02:16
@msyyc
Yuchao Yan (msyyc) merged commit 7d90591 into feature/lintdiff-migration-new Oct 10, 2026
3 checks passed
@msyyc
Yuchao Yan (msyyc) deleted the docs/lintdiff-publication-command-safety branch October 10, 2026 02:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant