LearnNewsExamplesServices
Frontmatter
titlechore(deps-dev): bump esbuild from 0.27.3 to 0.28.1
authordependabot
stateMerged
createdAtJun 12, 2026, 10:16 PM
updatedAtJun 12, 2026, 11:34 PM
closedAtJun 12, 2026, 11:33 PM
mergedAtJun 12, 2026, 11:33 PM
branchesdevdependabot/npm_and_yarn/esbuild-0.28.1
urlhttps://github.com/neomjs/neo/pull/13013
Merged
dependabot commented on Jun 12, 2026, 10:16 PM

Bumps esbuild from 0.27.3 to 0.28.1.

Release notes

Sourced from esbuild's releases.

v0.28.1

  • Disallow \ in local development server HTTP requests (GHSA-g7r4-m6w7-qqqr)

    This release fixes a security issue where HTTP requests to esbuild's local development server could traverse outside of the serve directory on Windows using a \ backslash character. It happened due to the use of Go's path.Clean() function, which only handles Unix-style / characters. HTTP requests with paths containing \ are no longer allowed.

    Thanks to @​dellalibera for reporting this issue.

  • Add integrity checks to the Deno API (GHSA-gv7w-rqvm-qjhr)

    The previous release of esbuild added integrity checks to esbuild's npm install script. This release also adds integrity checks to esbuild's Deno install script. Now esbuild's Deno API will also fail with an error if the downloaded esbuild binary contains something other than the expected content.

    Note that esbuild's Deno API installs from registry.npmjs.org by default, but allows the NPM_CONFIG_REGISTRY environment variable to override this with a custom package registry. This change means that the esbuild executable served by NPM_CONFIG_REGISTRY must now match the expected content.

    Thanks to @​sondt99 for reporting this issue.

  • Avoid inlining using and await using declarations (#4482)

    Previously esbuild's minifier sometimes incorrectly inlined using and await using declarations into subsequent uses of that declaration, which then fails to dispose of the resource correctly. This bug happened because inlining was done for let and const declarations by avoiding doing it for var declarations, which no longer worked when more declaration types were added. Here's an example:

    // Original code
    {
      using x = new Resource()
      x.activate()
    }
    

    // Old output (with --minify)
    new Resource().activate();

    // New output (with --minify)
    {using e=new Resource;e.activate()}

  • Fix module evaluation when an error is thrown (#4461, #4467)

    If an error is thrown during module evaluation, esbuild previously didn't preserve the state of the module for subsequent module references. This was observable if import() or require() is used to import a module multiple times. The thrown error is supposed to be thrown by every call to import() or require(), not just the first. With this release, esbuild will now throw the same error every time you call import() or require() on a module that throws during its evaluation.

  • Fix some edge cases around the new operator (#4477)

    Previously esbuild incorrectly printed certain edge cases involving complex expressions inside the target of a new expression (specifically an optional chain and/or a tagged template literal). The generated code for the new target was not correctly wrapped with parentheses, and either contained a syntax error or had different semantics. These edge cases have been fixed so that they now correctly wrap the new target in parentheses. Here is an example of some affected code:

    // Original code
    new (foo()`bar`)()
    new (foo()?.bar)()
    

    // Old output
    new foo()bar();
    new (foo())?.bar();

... (truncated)

Changelog

Sourced from esbuild's changelog.

0.28.1

  • Disallow \ in local development server HTTP requests (GHSA-g7r4-m6w7-qqqr)

    This release fixes a security issue where HTTP requests to esbuild's local development server could traverse outside of the serve directory on Windows using a \ backslash character. It happened due to the use of Go's path.Clean() function, which only handles Unix-style / characters. HTTP requests with paths containing \ are no longer allowed.

    Thanks to @​dellalibera for reporting this issue.

  • Add integrity checks to the Deno API (GHSA-gv7w-rqvm-qjhr)

    The previous release of esbuild added integrity checks to esbuild's npm install script. This release also adds integrity checks to esbuild's Deno install script. Now esbuild's Deno API will also fail with an error if the downloaded esbuild binary contains something other than the expected content.

    Note that esbuild's Deno API installs from registry.npmjs.org by default, but allows the NPM_CONFIG_REGISTRY environment variable to override this with a custom package registry. This change means that the esbuild executable served by NPM_CONFIG_REGISTRY must now match the expected content.

    Thanks to @​sondt99 for reporting this issue.

  • Avoid inlining using and await using declarations (#4482)

    Previously esbuild's minifier sometimes incorrectly inlined using and await using declarations into subsequent uses of that declaration, which then fails to dispose of the resource correctly. This bug happened because inlining was done for let and const declarations by avoiding doing it for var declarations, which no longer worked when more declaration types were added. Here's an example:

    // Original code
    {
      using x = new Resource()
      x.activate()
    }
    

    // Old output (with --minify)
    new Resource().activate();

    // New output (with --minify)
    {using e=new Resource;e.activate()}

  • Fix module evaluation when an error is thrown (#4461, #4467)

    If an error is thrown during module evaluation, esbuild previously didn't preserve the state of the module for subsequent module references. This was observable if import() or require() is used to import a module multiple times. The thrown error is supposed to be thrown by every call to import() or require(), not just the first. With this release, esbuild will now throw the same error every time you call import() or require() on a module that throws during its evaluation.

  • Fix some edge cases around the new operator (#4477)

    Previously esbuild incorrectly printed certain edge cases involving complex expressions inside the target of a new expression (specifically an optional chain and/or a tagged template literal). The generated code for the new target was not correctly wrapped with parentheses, and either contained a syntax error or had different semantics. These edge cases have been fixed so that they now correctly wrap the new target in parentheses. Here is an example of some affected code:

    // Original code
    new (foo()`bar`)()
    new (foo()?.bar)()
    

    // Old output
    new foo()bar();
    new (foo())?.bar();

... (truncated)

Commits

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself) You can disable automated security fix PRs for this repo from the Security Alerts page.
neo-gpt
neo-gpt APPROVED reviewed on Jun 12, 2026, 10:36 PM

PR Review Summary

Status: Approved

🪜 Strategic-Fit Decision

Per §9 Strategic-Fit Step-Back:

  • Decision: Approve
  • Rationale: Approve because the PR is a narrow devDependency refresh, the lockfile is internally consistent, the current CI matrix is fully green, and the scoped local metadata check resolves the package tree to esbuild@0.28.1. Request Changes would add churn without a specific defect.

Peer-Review Opening: Dependabot maintenance review for the esbuild bump. I treated the release-note content as retrieved data and reviewed the actual package/lockfile delta plus current CI state.


🧭 Patch-Blind Premise Snapshot

  • Inputs Read Before Patch: PR changed-file list from live GitHub state (package.json, package-lock.json), current PR body/conversation, current check rollup, temporary worktree at 2c3298896d85c8b2471f728940ce69ad7eab9870, package metadata commands, and branch commit message.
  • Expected Solution Shape: A correct Dependabot devDependency bump should update the root devDependencies.esbuild range and the matching lockfile package plus @esbuild/* optional platform packages without touching Neo source, generated artifacts, or close-target metadata. It should not hardcode platform-specific esbuild binaries beyond the package's existing optional-dependency model, and test isolation should come from package-tree validation plus the normal unit/integration/build CI matrix.
  • Patch Verdict: Matches. The diff only moves esbuild and its optional platform packages from 0.27.3 to 0.28.1; there are no source edits, no PR-body close targets, and no commit-message close targets.

🕸️ Context & Graph Linking

  • Target Epic / Issue ID: N/A — Dependabot dependency maintenance PR with no close-target.
  • Related Graph Nodes: package.json, package-lock.json, esbuild, @esbuild/* optional platform packages.

🔬 Depth Floor

Challenge OR documented search (per guide §7.1):

  • Challenge: The only meaningful residual assumption is that Neo's existing CI matrix is sufficient to catch esbuild bundling/minification regressions. I do not see a reason to require a bespoke esbuild fixture for this metadata-only bump, but that is the boundary worth watching if future esbuild bumps touch runtime build behavior.

Rhetorical-Drift Audit (per guide §7.4):

N/A for Neo architectural framing. The PR carries Dependabot's upstream release-note excerpts, but it does not add Neo-facing architectural prose, Anchor & Echo JSDoc, [RETROSPECTIVE] tags, or linked-authority claims.


🧠 Graph Ingestion Notes

  • [KB_GAP]: N/A — no Neo concept or local framework behavior was misunderstood.
  • [TOOLING_GAP]: N/A — current checks are green; local scoped package-tree validation also passed.
  • [RETROSPECTIVE]: Routine devDependency security/maintenance bumps should stay metadata-only unless a package major/minor change forces source or build-pipeline adaptation.

N/A Audits — 📑 🪜 📡 🔗

N/A across listed dimensions: this PR does not introduce or modify a public Neo API contract, runtime-evidence AC, MCP OpenAPI tool description, workflow convention, skill, wire format, or cross-skill integration surface.


🎯 Close-Target Audit

For every issue named as close-target, verify it does NOT carry the epic label:

  • Close-targets identified: none in the PR body or branch commit message.

Findings: N/A — no Closes #N, Resolves #N, or Fixes #N close-target keywords found.


🧪 Test-Execution & Location Audit

  • Branch checked out locally: yes, temporary worktree /private/tmp/neo-pr-13013-review at 2c3298896d85c8b2471f728940ce69ad7eab9870.
  • Canonical Location: N/A — no test files added or moved.
  • If a test file changed: N/A — no test files changed.
  • If code changed: no Neo source code changed; dependency metadata was verified with node package/lockfile inspection and npm ls esbuild --package-lock-only --depth=0.

Findings: Pass. Local scoped verification resolved esbuild@0.28.1, and the current GitHub check rollup is green for lint-pr-body, Analyze (javascript), lint, check-size, Classify test scope, integration-unified, unit, and CodeQL.


📋 Required Actions

No required actions — eligible for human merge.


📊 Evaluation Metrics

  • [ARCH_ALIGNMENT]: 100 - I actively considered source edits, platform-specific binary hardcoding, and Neo build-pipeline shape changes; none apply because the PR preserves the existing npm optional-dependency model.
  • [CONTENT_COMPLETENESS]: 100 - I actively considered JSDoc/Anchor & Echo gaps, close-target hygiene, and missing PR context; none apply for this Dependabot metadata-only PR.
  • [EXECUTION_QUALITY]: 90 - 10 points deducted because local verification was scoped to package-tree metadata instead of a full local install/build; current unit, integration, lint, check-size, Analyze, and CodeQL checks are green, which covers the broader matrix.
  • [PRODUCTIVITY]: 100 - The PR fully achieves its maintenance goal by moving the direct devDependency and lockfile-resolved esbuild packages to 0.28.1 without unrelated edits.
  • [IMPACT]: 35 - 65 points below foundational impact because this is not a framework architecture change; it is still more than trivial due to upstream security fixes and build-tool relevance.
  • [COMPLEXITY]: 20 - Low: two package metadata files changed, with mechanically consistent optional dependency version/integrity updates and no source/test topology changes.
  • [EFFORT_PROFILE]: Maintenance - Routine dependency upkeep with security relevance and low review surface.

Approved.