-
Notifications
You must be signed in to change notification settings - Fork 0
Complete packaging as VSIX with the CLI as a binary #2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…pdate README with new features and commands
|
Caution Review failedThe pull request is closed. WalkthroughThis update introduces several project-wide changes: it rebrands the extension from "deeproots" to "WildestAI" across metadata, documentation, and code. The README is rewritten for WildestAI, a GPLv3 license file is added, and Changes
Sequence Diagram(s)sequenceDiagram
participant VSCode
participant Extension
participant WildBinary
VSCode->>Extension: Activate extension
Extension->>VSCode: Register commands (wildestai.helloWorld, wildestai.generate)
VSCode->>Extension: User runs "wildestai.generate"
Extension->>Extension: Determine mode (dev/prod)
alt Development Mode
Extension->>WildBinary: Spawn Python venv "wild"
else Production Mode
Extension->>WildBinary: Spawn packaged "wild" binary (platform-specific)
end
WildBinary-->>Extension: Returns output
Extension->>VSCode: Show progress, log output, send notification
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Possibly related PRs
Note ⚡️ Unit Test Generation is now available in beta!Learn more here, or try it out under "Finishing Touches" below. 📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (2)
✨ Finishing Touches
🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 5
🧹 Nitpick comments (1)
.vscodeignore (1)
15-16: Redundant ignore pattern
*.envalready covers.envand.env.local; you can delete line 15 to keep the list concise.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (2)
media/logo-transparent.pngis excluded by!**/*.pngpackage-lock.jsonis excluded by!**/package-lock.json
📒 Files selected for processing (6)
.gitignore(1 hunks).vscodeignore(1 hunks)LICENSE(1 hunks)README.md(2 hunks)package.json(2 hunks)src/extension.ts(4 hunks)
🔇 Additional comments (5)
.gitignore (1)
8-10: Verify build-time availability ofbin/artifactsThe new rule ignores
bin/, yet the VSIX relies on that directory (see.vscodeignoreandpackage.json#bin).
Unless the CI pipeline copies the platform binaries intobin/aftergit clone, the VSIX will be produced without the CLI and fail at runtime.Consider adding an explicit build step (or Git LFS) that populates
bin/beforevsce package, or commit small shims that download the real binaries duringpostinstall.package.json (1)
8-9: Engine range may exclude many usersRequiring
^1.99.0means the extension will not install on the current stable VS Code (1.92 as of Aug-2025).
Double-check if you really need APIs introduced after 1.92; otherwise lower the floor.README.md (1)
18-24: Version requirement inconsistent withpackage.jsonREADME states “VS Code 1.70.0 or later” while
package.json.engines.vscoderequires^1.99.0. Align the two to avoid user confusion.src/extension.ts (2)
35-35: LGTM! Consistent rebranding implementation.The rebranding from "deeproots" to "Wildest AI" is consistently applied across activation messages and command IDs. The command naming follows VS Code conventions with lowercase identifiers.
Also applies to: 40-40, 43-43, 48-48
140-140: LGTM! Consistent CLI name updates.The error messages and notifications have been properly updated to reference the new "wild" CLI name and "Wildest AI" branding, maintaining consistency throughout the codebase.
Also applies to: 149-149, 173-173
|
Note Unit test generation is an Early Access feature. Expect some limitations and changes as we gather feedback and continue to improve it. Generating unit tests... This may take up to 20 minutes. |
|
✅ UTG Post-Process Complete No new issues were detected in the generated code and all check runs have completed. The unit test generation process has completed successfully. |
|
Creating a PR to put the unit tests in... The changes have been created in this pull request: View PR |
…Extension Marketplace.
Summary by CodeRabbit
New Features
Documentation
Chores
Refactor