Skip to content

fix(devbox): fixed log flashing on some terminals - #78

Merged
dines-rl merged 1 commit into
mainfrom
dines/fix-flashing
Jan 27, 2026
Merged

fix(devbox): fixed log flashing on some terminals#78
dines-rl merged 1 commit into
mainfrom
dines/fix-flashing

Conversation

@dines-rl

Copy link
Copy Markdown
Contributor

Description

Note: PR titles should follow Conventional Commits format (e.g., feat(devbox): add support for custom env vars or fix(snapshot): resolve pagination issue) as they are used for automatic release notes generation.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Code refactoring
  • Performance improvement
  • Test updates

Related Issues

Closes #

Changes Made

Testing

  • I have tested locally
  • I have added/updated tests
  • All existing tests pass

Checklist

  • My code follows the code style of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have updated the documentation accordingly
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published

Screenshots (if applicable)

Additional Notes

@dines-rl
dines-rl merged commit 8107796 into main Jan 27, 2026
14 checks passed
@dines-rl
dines-rl deleted the dines/fix-flashing branch January 27, 2026 18:25
dines-rl pushed a commit that referenced this pull request Jan 27, 2026
🤖 I have created a release *beep* *boop*
---


## [1.7.1](v1.7.0...v1.7.1)
(2026-01-27)


### Bug Fixes

* **devbox:** fixed log flashing on some terminals
([#78](#78))
([8107796](8107796))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
mrw-rl added a commit that referenced this pull request Jun 8, 2026
## What

Bumps the pnpm `hono` override `4.12.18` → `4.12.24` to clear four
medium Dependabot alerts (#76#79), all fixed in hono **4.12.21**:

| Alert | GHSA | Issue |
|-------|------|-------|
| #76 | `GHSA-f577-qrjj-4474` | JWT middleware accepts any
`Authorization` scheme, not just Bearer |
| #77 | `GHSA-xrhx-7g5j-rcj5` | IP restriction bypasses static deny
rules for non-canonical IPv6 |
| #78 | `GHSA-2gcr-mfcq-wcc3` | `app.mount()` strips prefix using
undecoded path → mis-routing on percent-encoded paths |
| #79 | `GHSA-3hrh-pfw6-9m5x` | Cookie helper does not sanitize
`sameSite`/`priority` → Set-Cookie injection |

## Why this approach

`hono` is purely transitive (`@modelcontextprotocol/sdk` →
`@hono/node-server`) and already governed by the pnpm `overrides` block,
so a single pin bump is the clean fix. Targeting `4.12.24` (latest in
the same minor line) rather than the minimum `4.12.21` — all patch-level
releases, a superset of the fixes, zero breaking surface.

Peer ranges accept it: `@hono/node-server` wants `^4`, the MCP SDK wants
`^4.11.4`. Lockfile resolves `hono@4.12.24` at every site with no
stragglers below the patched floor.

## Verification

- `pnpm run build` (tsc) → clean
- Tests: 800/804 pass. The 4 failures are the live-API `scp`/`rsync` e2e
suite failing on `401` (no API key in CI/sandbox) — unrelated to this
change.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.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