chore(release): v1.24.1 - #276
Merged
Merged
Conversation
Local-AI reliability (desktop Ollama/LM Studio/vLLM discovery, misleading browser status badge), the AI heuristic-fallback foundation + Outline/ Character/World/Plot-Board generators, real self-hosted LanguageTool integration, an 18-PR Dependabot backlog triage (AI SDK v4 family, biome 2.5.x, dev-tooling/Babel 8, plus 15 straightforward bumps), the Issue #60 vendor-fork audit closeout, and security/build hardening (persist-credentials CWE-522, ltr/rtl Tailwind fix, 3 ineffective dynamic imports, 2 CWE-209 fixes). - Version bump 1.24.0 -> 1.24.1 (package.json, src-tauri/Cargo.toml, src-tauri/tauri.conf.json, src-tauri/Cargo.lock, public/sw.js APP_VERSION via the existing sync scripts) - CHANGELOG.md [Unreleased] -> [1.24.1] - 2026-07-28 - README.md badges + metrics refreshed (2844 keys x 19 locales, 525 test files), AUDIT.md quality-gate line and follow-up chain updated, TODO.md v1.24.1 section finalized with the full Dependabot/CodeRabbit outcome - graphify-out/GRAPH_REPORT.md regenerated (AST-only, no API cost) Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (4)
🚧 Files skipped from review as they are similar to previous changes (4)
📝 WalkthroughWalkthroughThe release was updated from v1.24.0 to v1.24.1 across application metadata, service-worker caches, Tauri configuration, release documentation, audit records, project metrics, and the generated graph report. Changesv1.24.1 Release Metadata Refresh
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 4
🧹 Nitpick comments (1)
src-tauri/Cargo.toml (1)
3-3: 🩺 Stability & Availability | 🔵 TrivialVerify the desktop release artifacts for both Tauri metadata changes.
After these Rust/Tauri changes, dispatch
tauri-build.ymland confirm it reachesFinished N bundles.
src-tauri/Cargo.toml#L3-L3: verify the updated Rust package version builds successfully.src-tauri/tauri.conf.json#L4-L4: verify the updated Tauri configuration produces the expected desktop bundles.As per coding guidelines, after any Rust/Tauri change, dispatch
tauri-build.ymland verify that it reachesFinished N bundles.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@src-tauri/Cargo.toml` at line 3, After the Rust/Tauri metadata changes, dispatch tauri-build.yml and verify it reaches “Finished N bundles”. Validate the updated Rust package version at src-tauri/Cargo.toml:3-3 and the updated Tauri configuration at src-tauri/tauri.conf.json:4-4; neither site requires a further code change if the desktop artifacts build successfully.Source: Coding guidelines
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@CHANGELOG.md`:
- Around line 14-15: Update the Dependabot backlog count in the changelog entry
to 19 PRs, matching the authoritative totals documented in AUDIT.md and TODO.md.
Leave the surrounding release description unchanged.
In `@graphify-out/GRAPH_REPORT.md`:
- Around line 42-84: Replace the _COMMUNITY_* links in GRAPH_REPORT.md with
stable anchors that resolve within the report, or update the graphify-update
flow to preserve and commit the corresponding community pages under
graphify-out/wiki. Ensure every generated navigation target remains resolvable
in the committed artifact after scripts/graphify-update.mjs removes the wiki
directory.
- Around line 404-405: Reconcile community node-count semantics in the report
generator so the Communities section and Knowledge Gaps entries use the same
underlying count for each community, including Community 22 and the thin
communities listed here. Update the relevant community-counting logic or
regenerate GRAPH_REPORT.md from corrected output, and verify no conflicting
counts remain.
In `@README.md`:
- Around line 660-662: Update the “Current test metrics” heading in README.md
from 2026-06-16 to 2026-07-28, or to the actual date when these v1.24.1 metrics
were measured, while leaving the metric values unchanged.
---
Nitpick comments:
In `@src-tauri/Cargo.toml`:
- Line 3: After the Rust/Tauri metadata changes, dispatch tauri-build.yml and
verify it reaches “Finished N bundles”. Validate the updated Rust package
version at src-tauri/Cargo.toml:3-3 and the updated Tauri configuration at
src-tauri/tauri.conf.json:4-4; neither site requires a further code change if
the desktop artifacts build successfully.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: faf4526f-bcae-4312-926d-8a2548f8ea9f
⛔ Files ignored due to path filters (1)
src-tauri/Cargo.lockis excluded by!**/*.lock
📒 Files selected for processing (9)
AUDIT.mdCHANGELOG.mdREADME.mdTODO.mdgraphify-out/GRAPH_REPORT.mdpackage.jsonpublic/sw.jssrc-tauri/Cargo.tomlsrc-tauri/tauri.conf.json
🏁 CodeAnt Quality Gate ResultsCommit: ✅ Overall Status: PASSEDQuality Gate Details
|
- CHANGELOG.md/AUDIT.md/TODO.md: the Dependabot batch count was inconsistent (18 vs 19) and internally miscounted (a "remaining 5" bucket that actually listed 6 sub-items). Recounted directly from GitHub: 16 merged + 4 closed as superseded = 20 total, now consistent across all three files. - README.md: "Current test metrics" heading still said 2026-06-16 next to the freshly-updated v1.24.1 numbers -- corrected to 2026-07-28. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
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.
Summary
Cuts the v1.24.1 patch release, closing out this session's full body of work:
persist-credentials: falseacross 7 CI workflows (CWE-522), 2 CWE-209 fixes (stale-request race inAiProviderCard, unsafe error-message passthrough inollamaService.ts), invalidltr/rtlTailwind variant fix, 3 ineffective dynamic imports converted to static.What's in this PR specifically
1.24.0→1.24.1acrosspackage.json,src-tauri/Cargo.toml/Cargo.lock,tauri.conf.json,public/sw.js(via the existing sync scripts).CHANGELOG.md:[Unreleased]→[1.24.1] - 2026-07-28.README.mdbadges/metrics refreshed (2844 i18n keys × 19 locales, 525 test files).AUDIT.mdquality-gate line + follow-up chain updated;TODO.mdv1.24.1 section finalized.graphify-out/GRAPH_REPORT.mdregenerated (AST-only).Test plan
pnpm run lint— cleanpnpm run typecheck— cleanpnpm run i18n:check— 2844 keys × 19 locales, all in paritymain🤖 Generated with Claude Code
Summary by CodeRabbit
Documentation
Maintenance