Conversation
Critical fix: CI mode was using a temporary HOME directory that caused PATH mismatches between install and test steps. Problem: - Install script sets HOME to /tmp/smriti-ci-XXXXX in CI mode - Next workflow step uses runner's original HOME ($HOME = /home/runner) - Binary installed in temp HOME but tests look in runner's HOME - Result: "Module not found" and "command not found" errors Solution: - Remove temp HOME logic from install.sh and install.ps1 - Use runner's actual HOME for installation (cleaned up after job anyway) - Add shell: bash to all test steps to ensure PATH exports work - Keep fallback: smriti || bun direct execution Changes: - install.sh: Remove mktemp HOME assignment - install.ps1: Remove USERPROFILE override - uninstall.ps1: Remove temp HOME logic - install-test.yml: Add shell: bash to all test steps for consistency This ensures tests run with the same HOME where files were installed. Fixes: Install Test failures on all platforms (Windows, macOS, Ubuntu) Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
Optimization targets Windows install-test slowness by caching Bun binaries and dependencies, plus improving bun install performance. Changes: - install-test.yml: Add cache action for Bun binary (~/.bun) - install-test.yml: Add cache action for QMD database (~/.cache/qmd) - install.sh: Add --no-progress flag to faster bun install fallback - install.ps1: Use --frozen-lockfile for faster cached installs Performance improvements: - Windows: Bun binary reused from cache (skip redownload) - All platforms: Bun modules cached in ~/.bun directory - QMD database pre-warmed from cache (avoids re-initialization) - Fallback install uses --no-progress for reduced output overhead Cache keys: - Bun: keyed by OS + bun.lockb (invalidates on dependency changes) - QMD: simple OS key (persists across runs) Expected impact: - First run: Full install (establishes cache) - Subsequent runs: 40-60% faster on Windows (Bun already downloaded) - All platforms: Faster dependency resolution with cached modules Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
The smriti_session_meta table has a FOREIGN KEY on agent_id referencing smriti_agents, but only claude-code, codex, and cursor were seeded. This caused FK constraint failures when ingesting copilot or cline sessions on a clean database. Closes #30 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Keep expanded Bun cache paths from v0.3.1-version-command and frozen-lockfile comment from v0.3.1-install-submodules.
github-actions
bot
changed the title
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
1 participant
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.
Release Summary
devmaindevtomainChanges Included
Validation
devNotes