Skip to content

fix(audit): resolve linting and syntax issues, update actions#10

Merged
OgeonX-Ai merged 2 commits into
mainfrom
fix/audit-sweep
Jul 3, 2026
Merged

fix(audit): resolve linting and syntax issues, update actions#10
OgeonX-Ai merged 2 commits into
mainfrom
fix/audit-sweep

Conversation

@OgeonX-Ai

Copy link
Copy Markdown
Contributor

This PR fixes linting errors and logic bugs found during the audit-fix process. Changes include updating deprecated Node.js 20 action majors to v6, fixing yaml syntax errors, and resolving PSScriptAnalyzer warnings and parse errors.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 9a172cd6e3

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

param([string]$Command, [string[]]$Args = @())
Write-Log "Running: $Command $($Args -join ' ')"
& $Command @Args
param([string]$Command, [string[]]$Arguments = @())

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Preserve the wrapper parameter name used by callers

The operator still calls these helpers with -Args (for example Invoke-Checked -Command "codex" -Args ...), but this rename removes the Args parameter; -Args is not a valid abbreviation of -Arguments, so non-dry-run processing fails with “parameter cannot be found” before Codex or verification can run. Either keep an Args parameter/alias or update every call site to use -Arguments.

Useful? React with 👍 / 👎.

Cover the highest-blast-radius logic in the org-mutation engine:
- Assert-SafeChangeSet sensitive-path and diff-budget guards
- Get-ChangedFile porcelain parsing (renames, quoting, dedupe)
- Search-Issue GraphQL request construction
- Autopilot.Common helpers (Get-RepoName, Invoke-GhJson, Get-LogTail)

Operator functions are lifted via AST so no live gh/git runs on load.
Add tests/run-tests.ps1 single-command runner and wire Pester into CI.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@OgeonX-Ai OgeonX-Ai enabled auto-merge (squash) July 3, 2026 14:58
@OgeonX-Ai OgeonX-Ai merged commit 6f363b9 into main Jul 3, 2026
4 checks passed
@OgeonX-Ai OgeonX-Ai deleted the fix/audit-sweep branch July 3, 2026 15:20
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