Skip to content

CLI perf: reduce deps#22652

Merged
thdxr merged 27 commits intodevfrom
cli-perf/reduce-deps
Apr 16, 2026
Merged

CLI perf: reduce deps#22652
thdxr merged 27 commits intodevfrom
cli-perf/reduce-deps

Conversation

@thdxr
Copy link
Copy Markdown
Member

@thdxr thdxr commented Apr 15, 2026

Summary

  • Refactor plugin dependency installation to use shared npm service
  • Move TUI config test files from src/ to test/ directory
  • Simplify config handling and reduce dependencies

Changes

  • core: refactor plugin dependency installation to use shared npm service
  • Update npm service imports across CLI TUI modules
  • Relocate test fixtures and config tests to proper test directory
  • Various import path updates for consistency

Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

@thdxr thdxr enabled auto-merge (squash) April 15, 2026 18:22
thdxr added 7 commits April 15, 2026 17:39
Moved Npm service to shared package to enable consistent dependency management

across all components. Plugin dependencies are now installed with proper

version targeting and concurrent install deduplication via Flock.

Changes from user perspective:

- Faster TUI startup by removing unnecessary Instance.provide wrapper

- More reliable plugin dependency installation with proper serialization

- Cleaner config loading without redundant worktree parameter requirements
The TUI config files (tui.ts, tui-schema.ts, tui-migrate.ts, console-state.ts) are
source files used by the application, not test files. Moved them to the proper
location in src/cli/cmd/tui/config/ and updated all imports.
@thdxr thdxr force-pushed the cli-perf/reduce-deps branch from d8fe9ff to ba34400 Compare April 15, 2026 21:39
thdxr added 19 commits April 15, 2026 17:51
- Add tracing spans to instance bootstrap initialization for easier debugging of startup issues
- Make git branch discovery non-blocking so workspace loads faster without waiting for VCS state
Removed the managed tui config directory feature that allowed external
tools to override project-level tui.json settings. Users now only need
to configure their TUI settings through .opencode/tui.json in their
project root or the global config directory.
tui: remove stray debug console.log from thread startup
@thdxr thdxr disabled auto-merge April 16, 2026 06:03
@thdxr thdxr merged commit 675a46e into dev Apr 16, 2026
9 of 10 checks passed
@thdxr thdxr deleted the cli-perf/reduce-deps branch April 16, 2026 06:03
jerome-benoit pushed a commit to jerome-benoit/opencode that referenced this pull request Apr 16, 2026
xywsxp pushed a commit to xywsxp/opencode that referenced this pull request Apr 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant