docs(changelog): document Store lane, reducer governance, and merged batch - #528
docs(changelog): document Store lane, reducer governance, and merged batch#528adamgell wants to merge 1 commit into
Conversation
…batch Adds Unreleased entries for the merges the previous update predates: the Microsoft Store evidence lane (#518), Reducer Framework v1 governance (#519), Autopilot outside ESP (#450), Company Portal Windows LocalState (#460), bounded advanced server capture (#500), the widened Windows parser CI coverage with its two latent fixes (#523), and the quick-xml 0.41 migration (#471). Also corrects the install-action version range for #514. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
📝 WalkthroughWalkthroughThe Unreleased changelog now documents evidence analysis, parser coverage, Reducer Framework v1 governance, the quick-xml migration, and updated GitHub Actions versions. ChangesChangelog updates
Estimated code review effort: 1 (Trivial) | ~2 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
@coderabbitai full review |
✅ Action performedFull review finished. Your included review limit is currently reached under our Fair Usage Limits Policy. This review may still proceed through usage-based billing if eligible. Your next included review will be available in 55 minutes. |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 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`:
- Line 42: Update the Windows parser coverage changelog entry to remove the
incorrect `#460` reference, retaining `#523` as the Windows CI hardening issue
identifier (or replacing it with the dedicated parser-coverage issue if one
exists). Leave the entry’s description unchanged.
- Line 46: Update the GitHub Actions changelog entry to remove the unchanged
actions/checkout and actions/github-script upgrades, and report only
taiki-e/install-action changing from 2.85.5 to 2.85.7.
🪄 Autofix
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: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro Plus
Run ID: 96ee91b9-7b9a-48cb-a8cf-8e5977d24e99
📒 Files selected for processing (1)
CHANGELOG.md
|
|
||
| ### Build & CI | ||
|
|
||
| - **Windows parser coverage (#460 / #523)**: The ESP Windows CI job now runs the full parser-crate suite on Windows. The widened coverage surfaced and fixed two latent issues: fixture contract tests comparing native path separators against forward-slash manifest paths, and a unit test whose `tauri::test::mock_app()` usage made the whole Windows test executable unloadable (comctl32 v6 import without a manifest). |
There was a problem hiding this comment.
🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win
Correct the issue references for Windows parser coverage.
Line 11 already assigns #460 to Company Portal Windows LocalState. The PR objectives assign Windows CI hardening to #523. Remove #460 from this entry or replace it with the issue that tracks parser coverage.
🤖 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 `@CHANGELOG.md` at line 42, Update the Windows parser coverage changelog entry
to remove the incorrect `#460` reference, retaining `#523` as the Windows CI
hardening issue identifier (or replacing it with the dedicated parser-coverage
issue if one exists). Leave the entry’s description unchanged.
| - **Scoop bucket (#446)**: Bump Scoop manifest packaging to 1.5.1. | ||
| - **chrono floor (#501 / #417)**: Raise the `chrono` dependency floor to 0.4.35 so parser code that needs newer APIs is honest about the minimum version. | ||
| - **GitHub Actions**: Updated `actions/checkout` 4 → 7, `actions/github-script` 7.1.0 → 9.0.0, and `taiki-e/install-action` 2.85.2 → 2.85.5. | ||
| - **GitHub Actions**: Updated `actions/checkout` 4 → 7, `actions/github-script` 7.1.0 → 9.0.0, and `taiki-e/install-action` 2.85.2 → 2.85.7. |
There was a problem hiding this comment.
📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
🧩 Analysis chain
🏁 Script executed:
#!/usr/bin/env bash
set -euo pipefail
rg -n -C 2 'uses:\s*(actions/checkout|actions/github-script|taiki-e/install-action)@' \
.github/workflowsRepository: adamgell/cmtraceopen
Length of output: 6732
🏁 Script executed:
#!/usr/bin/env bash
set -euo pipefail
printf '%s\n' '--- relevant diff ---'
git diff --unified=3 -- CHANGELOG.md .github/workflows
printf '%s\n' '--- current relevant references ---'
rg -n 'uses:\s*(actions/checkout|actions/github-script|taiki-e/install-action)@' .github/workflowsRepository: adamgell/cmtraceopen
Length of output: 2315
🏁 Script executed:
#!/usr/bin/env bash
set -euo pipefail
printf '%s\n' '--- repository state ---'
git status --short
git rev-parse --show-toplevel
git log -1 --oneline
git rev-parse HEAD^ >/dev/null
printf '%s\n' '--- changelog entry ---'
sed -n '42,49p' CHANGELOG.md
printf '%s\n' '--- commit diff ---'
git diff --unified=3 HEAD^ HEAD -- CHANGELOG.md .github/workflows
printf '%s\n' '--- parent references ---'
git grep -n -E 'uses:\s*(actions/checkout|actions/github-script|taiki-e/install-action)@' HEAD^ -- .github/workflows || trueRepository: adamgell/cmtraceopen
Length of output: 7429
Correct the GitHub Actions changelog entry. actions/checkout and actions/github-script did not change in this diff. Only taiki-e/install-action changed, from 2.85.5 to 2.85.7.
🤖 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 `@CHANGELOG.md` at line 46, Update the GitHub Actions changelog entry to remove
the unchanged actions/checkout and actions/github-script upgrades, and report
only taiki-e/install-action changing from 2.85.5 to 2.85.7.
Source: Path instructions
|
Superseded by #530, which documents the same batch with a more complete set of entries. |
Unreleased-section refresh covering everything merged since the last changelog update: Store evidence lane (#518), Reducer Framework v1 governance (#519), Autopilot outside ESP (#450), Company Portal Windows LocalState (#460), bounded advanced server capture (#500), Windows CI hardening (#523), quick-xml 0.41 migration (#471), and the install-action range correction (#514).
🤖 Generated with Claude Code
Summary by CodeRabbit