chore: migrate dependencies from Node.js to Deno native#289
chore: migrate dependencies from Node.js to Deno native#289jaypatrick merged 12 commits intomainfrom
Conversation
- Replace @std/flags with @std/cli (parseArgs) - Remove unused @luca/cases import - Replace node:process with Deno.env.get() - Update deno.lock with new dependencies
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ✅ Deployment successful! View logs |
adblock-compiler | b0fe0a8 | Feb 11 2026, 10:24 PM |
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
Overview
This PR migrates CLI argument parsing from the deprecated @std/flags to @std/cli, removes unused @luca/cases dependency, and replaces Node.js process.env usage with native Deno.env.get() in PrismaStorageAdapter.
Concerns
- Unrelated file in PR: The
.github/agents/code-task-executor.agent.mdfile is completely unrelated to the dependency migration described in the PR title/description. This should either be removed from this PR or the PR description should be updated to reflect all changes. - The core migration changes (
parse→parseArgs,@std/flags→@std/cli/parse-args) are correct and the API surface appears compatible.
Verdict
Found: 0 critical, 1 important, 1 suggestion
Approve with nits — the dependency migration is correct, but the unrelated agent file should be addressed
There was a problem hiding this comment.
Pull request overview
Migrates CLI/runtime dependencies away from Node.js/npm-style and deprecated Deno std modules toward Deno-native equivalents, primarily updating argument parsing and environment access patterns.
Changes:
- Replaced
@std/flagsusage withparseArgsfrom@std/cli. - Removed unused
@luca/casesreferences from the Deno import map / lockfile. - Updated Prisma storage adapter to read
DATABASE_URLviaDeno.env.get()(removingnode:process).
Reviewed changes
Copilot reviewed 6 out of 7 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
src/storage/PrismaStorageAdapter.ts |
Removes node:process usage and updates env var lookup for DB connection. |
src/cli/CliApp.deno.ts |
Switches CLI arg parsing from @std/flags to @std/cli. |
src/cli/ArgumentParser.ts |
Switches shared arg parser from @std/flags to @std/cli. |
scripts/record-deployment.ts |
Switches script arg parsing from @std/flags to @std/cli. |
deno.json |
Updates import map (drops @std/flags / @luca/cases, adds @std/cli). |
deno.lock |
Lockfile updates reflecting dependency changes. |
.github/agents/code-task-executor.agent.md |
Adds an agent instruction file (new). |
|
I detected a CI failure. I have created a fix PR: #292 |
|
@copilot plesae fix the CI format check error |
|
@jaypatrick I've opened a new pull request, #293, to work on those changes. Once the pull request is ready, I'll request review from you. |
Co-authored-by: jaypatrick <1800595+jaypatrick@users.noreply.github.com>
chore: fix markdown formatting to comply with Deno formatter
Fix CI Failure for PR #289
|
@copilot bump the version to 0.12.0 |
|
@jaypatrick I've opened a new pull request, #294, to work on those changes. Once the pull request is ready, I'll request review from you. |
Co-authored-by: jaypatrick <1800595+jaypatrick@users.noreply.github.com>
chore: bump version to 0.12.0
Summary
Migrates dependencies from Node.js/npm to Deno native equivalents where possible.
Changes
@std/flags→@std/cli: Replaced deprecated@std/flagswith the modern@std/clipackage (usingparseArgsfrom@std/cli/parse-args)@luca/cases: Was declared in imports but never used in codenode:process: Replaced with nativeDeno.env.get()in PrismaStorageAdapterFiles Modified
deno.json- Updated import mapdeno.lock- Updated lockfilesrc/cli/ArgumentParser.ts- UsesparseArgssrc/cli/CliApp.deno.ts- UsesparseArgsscripts/record-deployment.ts- UsesparseArgssrc/storage/PrismaStorageAdapter.ts- UsesDeno.env.get()Testing