Repository navigation
[instructions] Sync instruction files with release v0.93.2 - #71598
Conversation
gh aw compile now runs built-in Actions/AW AST lint by default and fails on findings (release v0.93.2, PR #71349); cli-commands.md and debug-compilation.md had no mention of this or the --skip-lint escape hatch. Also trims a sentence in cli-commands.md duplicated by debug-compilation.md's existing severity coverage, to stay under the 400-line instruction-file limit. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
|
@copilot rename --skip-lint to --no-lint for consistency |
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Renamed the compile CLI flag to |
|
@copilot address the following outstanding work in one pass:
Push the necessary fixes, reply to each listed review thread and resolve it when addressed. Ignore feedback already answered or resolved. Use the pr-finisher skill and stop when only human review or CI remains; do not trigger CI. Sous-chef head: b0a6320
|
…93.2-78138123fc2e80c1 Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com>
Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com>
Merged latest |
|
🎉 This pull request is included in a new release. Release: |
Audit summary
Release baseline: v0.93.2 (published 2026-10-23T05:57:09Z). Local checkout is shallow (single commit), so release/change history was gathered via GitHub MCP
list_commits/get_commitinstead of localgit diff/git logagainst the tag.Documentation commits reviewed since release (5, excluding a blog post):
1233a55c— trimmeddebug-workflow-evidence.mdunder 400 lines (already compliant, no action)ddaf87da— FAQ clarification thatworkflow_callreusable workflows must live directly in.github/workflows/; this concerns GitHub Actions' native reuse mechanism, distinct from gh-aw's import-based shared components increate-shared-agentic-workflow.md— no inconsistency found88cc9828—feat: lint workflows during compilation by default(feat: lint workflows during compilation by default #71349, major/breaking) — addressed below9084a8c7— self-hosted runner proxy inheritance + Grype scan-completeness docs; infra/admin-facing content not currently mirrored into.github/aw/, and--grype/--runner-guardwere already undocumented there before this release (pre-existing gap, not a regression from this release — left out of scope)0489b24e— daily work-queue smoke workflows + Bash 3 validation; only touched generated.lock.ymlfiles, no.mdinstruction contentFiles changed
.github/aw/cli-commands.md(396 → 399 lines): addedgh aw compile --skip-lintto the command list and a note that compilation now runs built-in Actions/AW AST lint by default (including--no-emit/validate) and fails on findings, with--skip-lint(MCPskip_lint: true) rejected together with--dry-run. Also trimmed one sentence duplicated bydebug-compilation.md's own severity coverage to stay under the 400-line limit..github/aw/debug-compilation.md(253 → 130... actually 124 → 130 lines): added built-in AST lint findings to the existing "remains fatal" list and a short paragraph distinguishing default AST lint (new) from the opt-in zizmor/actionlint/poutine scanners, noting--skip-lintis rejected with--dry-run.Verified the
--skip-lintflag, its--dry-runincompatibility, and the MCPskip_lintparameter directly against source (pkg/cli/compile_config.go,pkg/cli/compile_compiler_setup_test.go,pkg/cli/mcp_tools_readonly.go) rather than relying solely on the changeset description.Size audit
wc -l .github/aw/*.md— all 79 files are under the 400-line target both before and after edits (max is nowcli-commands.mdat 399).Duplication audit
Removed one sentence from
cli-commands.md("Low-severity/informational notices are not blanket failures") that duplicateddebug-compilation.md's own detailed severity coverage, consistent with the index-file-should-reference-not-restate principle.Accuracy audit
Confirmed via source that
--skip-lint,skip_lint, and their--dry-runincompatibility exist as described in the release changeset (.changeset/compile-default-ast-lint.md). No other schema/safe-outputs drift found inpkg/workflow/compiler_types.go,pkg/workflow/safe_outputs_config.go, or the main workflow JSON schema relevant to these changes.Steering
Checked issue #71594 for
steer-tagged comments; none present.🤖 Generated with [Claude Code]((claude.com/redacted)