Skip to content

chore: refresh tui package lock#18573

Closed
AJV20 wants to merge 2 commits into
NousResearch:mainfrom
AJV20:chore/update-tui-package-lock
Closed

chore: refresh tui package lock#18573
AJV20 wants to merge 2 commits into
NousResearch:mainfrom
AJV20:chore/update-tui-package-lock

Conversation

@AJV20

@AJV20 AJV20 commented May 1, 2026

Copy link
Copy Markdown

Summary

  • Refreshes ui-tui/package-lock.json metadata from npm.
  • No source changes.

Test Plan

  • git diff --check
  • cd ui-tui && npm install --package-lock-only --ignore-scripts

Notes

  • npm reports one existing moderate vulnerability; this PR does not change dependency selections to address audit findings.

@alt-glitch alt-glitch added type/refactor Code restructuring, no behavior change comp/tui Terminal UI (ui-tui/ + tui_gateway/) P3 Low — cosmetic, nice to have labels May 1, 2026
@AJV20

AJV20 commented May 22, 2026

Copy link
Copy Markdown
Author

Updated this PR branch in 604e3ebd7 after the latest main merge.

What changed:

  • Merged current origin/main into chore/update-tui-package-lock.
  • Resolved conflicts where needed by preserving this PR's scoped behavior while keeping current upstream changes.

Verification:

  • git diff --check; cd ui-tui && npm install --package-lock-only --ignore-scripts → up to date

Current pushed head: 604e3ebd7710

@teknium1

Copy link
Copy Markdown
Contributor

This looks safe to close as already handled on current main by an automated hermes-sweeper review.

Evidence:

  • PR chore: refresh tui package lock #18573 now has no file diff: GitHub reports changed_files: 0, additions: 0, deletions: 0, and gh pr diff 18573 --name-only returns no paths.
  • The TUI-local lockfile this PR refreshed was superseded on main by a51a7b9b92b63b5f97afa0fc31c9cd349f04b5ec (fix(node/nix): consolidate workspace lockfile + update all consumers), which deleted ui-tui/package-lock.json and updated the root workspace lock.
  • Current package-lock.json includes ui-tui and ui-tui/packages/* in the root workspace lock at package-lock.json:12-16.

Thanks for the cleanup PR; the lockfile maintenance has been absorbed by the later workspace-lock consolidation.

@teknium1 teknium1 closed this Jun 10, 2026
@teknium1 teknium1 added the sweeper:implemented-on-main Sweeper: behavior already present on current main label Jun 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/tui Terminal UI (ui-tui/ + tui_gateway/) P3 Low — cosmetic, nice to have sweeper:implemented-on-main Sweeper: behavior already present on current main type/refactor Code restructuring, no behavior change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants