fix(memory): QMD path resolution + setup wizard improvements + skill scan#5
Merged
Conversation
…ath resolution
createRequire().resolve('@tobilu/qmd/package.json') fails with
ERR_PACKAGE_PATH_NOT_EXPORTED because QMD's exports map doesn't
include './package.json'. Switch to import.meta.resolve() which
uses ESM import conditions and resolves correctly without patching.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
… step Each adapter step (Discord/Slack/Telegram) now collects tokens inline instead of deferring to a separate "secrets" step. After saving the token to the vault, a connection test validates the credential against the platform API (Discord /users/@me, Slack auth.test, Telegram getMe). Tests fail gracefully with a warning — they don't block setup. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Replace one-shot token entry with a retry loop via collectAndTestToken(). On connection test failure, the user can re-enter the token or skip (saving the current token anyway). This prevents typos from requiring a full re-run of the setup wizard. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
stepHome() now receives which adapters were enabled during the current wizard run, skipping channel fetches for previously-enabled adapters that weren't selected this time. Standalone --step home still loads all enabled adapters. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
initializeWorkspace now accepts { skipQmd } option. The setup wizard
skips QMD on the initial workspace init (Step 1) and runs it during
the final re-initialization with a descriptive spinner so users know
what's happening.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Remove the duplicate initializeWorkspace call at the start of the wizard. Workspace init, MCP server registration, and QMD model download now run as the last step with a clear spinner message. No more lite/skipQmd option — initializeWorkspace always runs fully. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- S-1(setup): Sanitize Telegram token from error messages (URL path leak)
- L-1(setup): Replace magic string check with discriminated union
ConnectionTestResult { ok, message/error }
- S-2(setup): Skip signingSecret prompt when Slack token is empty
- R-1(init): Deduplicate resolveQmdEntrypoint — export from qmd.ts
- E-1(init): Surface QMD collection add failure as warning instead of
silently swallowing
- S-1(skills): Add process.exit(2) for danger results in stageSkill scan path
- S-2(skills): Use cleanupStaging() instead of raw rmSync in stageSkill catch
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- skill scan now accepts <ref> (owner/repo, URL) to scan before installing instead of requiring an installed skill name. --local flag for existing skills. - LLM advisory changed from default to opt-in (--llm) since it requires ACP session startup (~1min/skill). Static pattern scan is instant. - Fix DEFAULT_SCAN_MODEL from invalid 'gemini-3.1-pro' to 'pro' (Gemini CLI alias) - Add Skills section to README with scan/install/manage workflow Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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
QMD path resolution fix
createRequire().resolve('@tobilu/qmd/package.json')がERR_PACKAGE_PATH_NOT_EXPORTEDで失敗し、QMD インデックス更新が全滅していたimport.meta.resolve('@tobilu/qmd')+fileURLToPathに置き換えSetup wizard improvements
initializeWorkspaceを最後に1回だけ実行するようリファクタSkill scan enhancements
skill scanが<ref>(owner/repo, URL)を受け付け、インストール前にスキャンできるように変更。インストール済みスキルは--localで指定--llm)に変更(ACP セッション起動で ~1分/スキルかかるため)DEFAULT_SCAN_MODELを無効なgemini-3.1-proからpro(Gemini CLI エイリアス)に修正Test plan
geminiclaw skill scan anthropics/claude-agent-sdk-demosで静的スキャンが即完了することを確認🤖 Generated with Claude Code