fix: consolidated critical fixes — LFS, Windows, model input, SKILL.MD case, bootstrap, route selection#79
Open
rollroyces wants to merge 3 commits into
Open
Conversation
…penBMB#48) Git LFS budget exceeded, blocking all new clones. - Removed assets/ from LFS tracking in .gitattributes - Downloaded all LFS objects (gif, png, mp4, mov) from CDN - Re-added as regular git blobs - Closes OpenBMB#48
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Consolidated fix for 6 critical open issues. All changes are cherry-picked from existing open PRs that were never merged, plus a manual LFS budget fix.
Fixes
🚨 Issue #48 — Git LFS budget exceeded (blocks all clones)
.gitattributes(*.gif,*.png,*.mp4,*.mov, etc.)🚨 Issue #28 —
npm run devfails on Windows (spawn npm ENOENT)npm.cmdon Windows,npmotherwise{ shell: true }for Windows spawn🚨 Issue #21 — Model ID input only accepts 1 character
useStateto uncontrolleduseReffor the custom model ID input🟠 Issue #40 — Route UI cannot select configured models
🟡 Issue #37 — SKILL.MD filenames case-sensitive check fails
findSkillMdFile()helper that matches/^skill\.md$/iu"SKILL.md"references with case-insensitive lookup🟡 Issue #36 —
npm start/npm serverskips bootstrap configprestarthook inui/package.jsonto runbootstrap-pilotdeck-config.mjsNotes
catalog/providers.tsandcatalogProviders.tsafter PRs 74/75 inadvertently removed itTesting
npm run buildpassesSKILL.MD,skill.md,SKILL.mdcase variants