Skip to content

Conversation

@pingSubhajit
Copy link
Contributor

Problem

The deep-merge.ts utility file was not being copied to the user's project during CLI installation (unrag init). This caused runtime errors when code that depends on the deep merge functionality attempted to import it.

Solution

Added core/deep-merge.ts to the list of core files copied during the installation process in packages/unrag/cli/lib/registry.ts.

Changes

  • packages/unrag/cli/lib/registry.ts: Added the deep-merge.ts file to the copyRegistryFiles function so it gets installed alongside other core files
  • packages/unrag/test/init.test.ts: Added a test assertion to verify that deep-merge.ts is present after running unrag init
  • apps/web/app/install/install-wizard-client.tsx: Simplified the install wizard UI layout

Testing

  • Added test coverage to ensure the file is present after initialization
  • Existing tests continue to pass

@pingSubhajit pingSubhajit self-assigned this Jan 8, 2026
@vercel
Copy link

vercel bot commented Jan 8, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
unrag-web Ready Ready Preview, Comment Jan 8, 2026 6:12pm

@pingSubhajit pingSubhajit changed the base branch from main to release/v0.2.8 January 8, 2026 18:12
@pingSubhajit pingSubhajit merged commit b3cf999 into release/v0.2.8 Jan 8, 2026
3 checks passed
@pingSubhajit pingSubhajit deleted the fix/missing-deep-merge-function-after-installation branch January 8, 2026 18:14
pingSubhajit added a commit that referenced this pull request Jan 9, 2026
* fix: deep merge file not present after installation (#13)
* feat: robust logging for pdf:text-layer extractor (#14)
* docs: new supported runtimes page in the docs
* feat: Add unrag doctor command for installation validation and troubleshooting (#15)
* chore: add spec for eval harness feature
* feat: Add reranker battery with Cohere and custom reranker support (#16)
* chore: bump patch version & updated new badge
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.

2 participants