fix(ci): build vscode-ail-chat on tag, not legacy vscode-ail#160
Merged
AlexChesser merged 6 commits intomainfrom Apr 21, 2026
Merged
fix(ci): build vscode-ail-chat on tag, not legacy vscode-ail#160AlexChesser merged 6 commits intomainfrom
AlexChesser merged 6 commits intomainfrom
Conversation
Tag pushes (vscode-v*) were packaging the legacy vscode-ail extension. Point the release workflow (working directory, cache path, dist download, VSIX artifact upload, release files, and marketplace publish) at vscode-ail-chat, which is the current extension.
Adds an Extension (chat) job to ci-extension.yml that runs compile, lint, and vitest against vscode-ail-chat. Also adds DOM.Iterable to the extension's tsconfig lib so the NodeList for-of loops in the webview tests type-check.
…code-ail-chat The legacy extension is retained as a code reference only; CI, the pre-commit hook, the README link, the /check command, and .gitignore now target vscode-ail-chat exclusively. The single extension CI job builds, lints, and runs vitest against vscode-ail-chat.
The .vscodeignore keeps the VSIX lean — only dist/, images/, package.json, and README.md ship (290 KB, 12 files vs. 366 KB, 56 files without it). The README satisfies vsce's marketplace-listing expectation and documents commands, config, and pipeline discovery for end users.
- Per-directory LICENSE files with full SPDX license text: - spec/ CC-BY-SA-4.0 - ail-core/ MPL-2.0 - ail/ AGPL-3.0-only - stub-llm/ MPL-2.0 (matches ail-core; test helper) - demo/ CC0-1.0 - vscode-ail-chat/ MIT (permissive to match VS Code marketplace convention) - Root LICENSE: navigation overview with the license table and per-folder pointers. Does not itself grant rights; the per-folder LICENSE files do. - Cargo.toml: declare `license =` for ail, ail-core, and stub-llm so the SPDX identifier travels with any published crate. - vscode-ail-chat/package.json: switch declared license from AGPL-3.0-only to MIT; update the extension README license note to match. - README license table: add rows for stub-llm/ and vscode-ail-chat/. Packaging the VSIX now bundles LICENSE.txt and vsce no longer warns about a missing license.
Rust 1.95's tightened collapsible_match lint flags the `if …is_err() { break; }`
inside the UserMessage match arm. Clippy's suggested match-guard fix would
leave the "send succeeds" case unhandled and break exhaustiveness, so use
`let Ok(()) = … else { break; }` instead — same semantics, no nested `if`.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Tag pushes (vscode-v*) were packaging the legacy vscode-ail extension.
Point the release workflow (working directory, cache path, dist download,
VSIX artifact upload, release files, and marketplace publish) at
vscode-ail-chat, which is the current extension.