Skip to content

Fix orphan removal on "gh aw remove"#11

Merged
dsyme merged 5 commits into
mainfrom
aw3
Aug 12, 2025
Merged

Fix orphan removal on "gh aw remove"#11
dsyme merged 5 commits into
mainfrom
aw3

Conversation

@dsyme

@dsyme dsyme commented Aug 12, 2025

Copy link
Copy Markdown
Collaborator

@dsyme dsyme merged commit ef9663b into main Aug 12, 2025
6 checks passed
@dsyme dsyme deleted the aw3 branch August 12, 2025 22:40
Copilot AI added a commit that referenced this pull request Feb 14, 2026
- Updated max-turns from 50 to 100
- Added all PR review safe output configurations:
  - update-pull-request
  - close-pull-request
  - create-pull-request-review-comment
  - submit-pull-request-review
  - resolve-pull-request-review-thread
  - push-to-pull-request-branch
  - add-reviewer
- Added comprehensive test requirements for each PR review safe output (#11-17)
- Updated output section to handle pass/fail/skip status
- Recompiled workflow successfully

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI added a commit that referenced this pull request Jun 27, 2026
- Change engine from codex to copilot with copilot-sdk: true to
  enable LSP support (LSP is Copilot-only per compiler validation)
- Add lsp.typescript frontmatter with typescript-language-server
  covering .js, .cjs, .mjs, .ts, .tsx extensions
- Add Node.js runtime (v20) required by the TypeScript LSP server
- Add test #11: use LSP document symbols to count functions in
  actions/setup/js/safe_output_helpers.cjs and report >=1 as pass
- Update issue body reference to include test #11 in results
- Recompile: lock.yml updated with lspServers settings.json payload,
  npm install --ignore-scripts typescript typescript-language-server,
  and NPM_CONFIG_MIN_RELEASE_AGE cooldown env var

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI added a commit that referenced this pull request Jun 27, 2026
…ilot

- Revert LSP changes from smoke-codex (engine change from codex to copilot,
  lsp frontmatter, node runtime v20, test #11 LSP TypeScript)
- Add TypeScript LSP server to smoke-copilot with lsp frontmatter and
  node runtime v20
- Add test #16 LSP TypeScript to smoke-copilot to count functions in
  actions/setup/js/safe_output_helpers.cjs
- Recompile both workflows

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
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