Skip to content

[codex] Sync ui-tui lockfile - #35449

Closed
Havokyn wants to merge 1 commit into
NousResearch:mainfrom
Havokyn:codex/sync-ui-tui-lockfile
Closed

[codex] Sync ui-tui lockfile#35449
Havokyn wants to merge 1 commit into
NousResearch:mainfrom
Havokyn:codex/sync-ui-tui-lockfile

Conversation

@Havokyn

@Havokyn Havokyn commented May 30, 2026

Copy link
Copy Markdown

Summary\n- syncs ui-tui package lock metadata from local workspace\n\n## Validation\n- not run; publish sweep only

@alt-glitch alt-glitch added P3 Low — cosmetic, nice to have comp/tui Terminal UI (ui-tui/ + tui_gateway/) dependencies Pull requests that update a dependency file labels May 30, 2026
@alt-glitch

Copy link
Copy Markdown
Collaborator

Duplicate of #18577 and #29605 — all three sync ui-tui/package-lock.json.

@teknium1

Copy link
Copy Markdown
Contributor

Thanks for the lockfile cleanup.

This automated hermes-sweeper review found that the requested update is already superseded on current main:

  • a51a7b9b92b63b5f97afa0fc31c9cd349f04b5ec (fix(node/nix): consolidate workspace lockfile + update all consumers) deleted ui-tui/package-lock.json and made the root workspace lockfile authoritative.
  • package.json:6-10 and package-lock.json:12-16 declare the root workspace structure including ui-tui and ui-tui/packages/*.
  • tests/hermes_cli/test_tui_npm_install.py:465-499 now enforces that workspace subdirectories, including ui-tui, do not contain their own package-lock.json.
  • This also matches the prior member note linking duplicate PRs fix: sync ui-tui lockfile for npm ci #18577 and chore(ui-tui): refresh package lock metadata #29605. The consolidation shipped in v2026.6.5.

@teknium1 teknium1 closed this Jul 13, 2026
@teknium1 teknium1 added the sweeper:implemented-on-main Sweeper: behavior already present on current main label Jul 13, 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/) dependencies Pull requests that update a dependency file P3 Low — cosmetic, nice to have sweeper:implemented-on-main Sweeper: behavior already present on current main

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants