Repository navigation
Conversation
Houdini already lived under skills/ but was left out of the app copy, the Xcode membership exceptions, and the first-launch seed list, so new installs never received it. Co-authored-by: Mohamed Saleh Zaied <engmsaleh@users.noreply.github.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This branch was successfully deployed
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.
References #110.
Houdini Essentials already lived at
skills/houdini-essentials, but the macOS app only shipped the other five skills. New installs never received it.The five bundled skills are shipped by three pieces that have to stay in sync:
leanring-buddy/skills/<id>/SKILL.md.membershipExceptions, so theSKILL.mdis not compiled as a source file. The existing Run Script build phase then copies every subdirectory ofleanring-buddy/skills/intoContents/Resources/skills/. That script did not need a change; it already copies whatever folders are present.SkillStore.bundledSkillNames, which is what first launch seeds into~/.skilly/skills/.Houdini now follows that same path. Id
houdini-essentials, display name Houdini Essentials, bundle idcom.sidefx.houdini,pointing_mode: always.Format
Compared with the other five bundled skills. Same single file (
SKILL.md), same required frontmatter fields, and the same body shape: Teaching Instructions, Curriculum (### Stage N:with Goals, Completion signals, and Next), and UI Vocabulary. Stage links chain correctly through all 9 stages. Teaching-instruction size (~2,910 tokens) and total size (~8,507 tokens) stay under the app limits (4,000 and 10,000). No content was rewritten.One extra frontmatter field is present only on Houdini:
last_verified: "2026-05-16". The Swift parser ignores unknown keys, so this does not block loading. It was left in place.Verification
project.pbxprojwith thepbxprojlibrary. The three targets load, object ids are unique, braces balance, andskills/houdini-essentials/SKILL.mdis inmembershipExceptions. Every exception path exists on disk.SKILL.mdfiles, includinghoudini-essentials.skills/houdini-essentials/SKILL.md. The seed list matches the six folders underleanring-buddy/skills/.Not verified
plutilis not installed in this environment. Linuxplistutilonly accepts XML or binary plists, so it cannot lint this OpenStep project file.xcodebuildandswiftcare not available.xcodebuildwas not run, because it invalidates macOS TCC permissions. The XCTest suite inleanring-buddyTestswas not run. A Mac build is still needed to confirm the skill shows up in the panel and auto-activates when Houdini is frontmost.Please do not merge until that Mac check is done.