Skip to content

docs: prepare release notes for v3.9.4#952

Merged
carlos-alm merged 2 commits into
mainfrom
release/3.9.4
Apr 17, 2026
Merged

docs: prepare release notes for v3.9.4#952
carlos-alm merged 2 commits into
mainfrom
release/3.9.4

Conversation

@carlos-alm
Copy link
Copy Markdown
Contributor

Summary

  • Add CHANGELOG entry for v3.9.4 (all commits since v3.9.3)
  • Update ROADMAP: bump current version, note v3.9.4 progress on Phase 8.3 (callback resolution heuristic)
  • Update BACKLOG last-updated date

Bump rationale: patch, not minor. Both feat: commits since v3.9.3 are internal improvements with no new CLI surface — feat(js-extractor) improves resolver accuracy (classified as Bug Fix, following the v3.9.1 precedent for similar consumption-tracking fixes), feat(native) ports structure computation to Rust (classified as Performance).

After merging: create a GitHub Release with tag `v3.9.4` to trigger the publish workflow, which handles version bumping, native binary builds, and npm publishing.

Test plan

  • CHANGELOG renders correctly on GitHub
  • ROADMAP checklist items match actual codebase state

@claude
Copy link
Copy Markdown

claude Bot commented Apr 17, 2026

Claude finished @carlos-alm's task —— View job


Reviewing PR: docs: prepare release notes for v3.9.4

Todo List:

  • Read changed files to understand documentation updates
  • Analyze changelog accuracy against actual commits since v3.9.3
  • Review roadmap and backlog changes
  • Assess version bump rationale (patch vs minor)
  • Apply 5-phase review framework
  • Provide final rating and recommendation

Root Cause Analysis

Problem Being Solved: Preparing comprehensive release documentation for v3.9.4, updating version references across documentation files, and providing clear rationale for the patch-level version bump.

Why This Approach: Pure documentation changes with no code impact - appropriate for a release preparation PR. The changes update version references and document completed work.

Risk Assessment: Minimal risk - documentation-only changes with no functional impact.

Backlog Compliance

  • Zero-dep: ✅ - No runtime dependencies added (documentation only)
  • Foundation-aligned: ✅ - No violation of core principles (pure documentation)
  • Problem-fit: N/A - Documentation maintenance, not feature development
  • Breaking: No - Purely additive documentation changes
  • Tier: N/A - Documentation maintenance outside tier system

Critical Review Findings

Changelog Accuracy

The v3.9.4 changelog entry accurately reflects all commits since v3.9.3:

Version Bump Rationale

The patch classification is justified:

Roadmap Updates

Documentation Consistency

  • BACKLOG.md last-updated date properly maintained (2026-04-17)
  • All version references consistently updated
  • No contradictory information across files

⚠️ Minor Observation

The changelog's performance section could more clearly distinguish between the feat(native) port (#937) and the pure performance optimizations (#939, #940) since feat: commits typically indicate new functionality, but this is classification preference rather than error.

Final Recommendation

  • Rating: ⭐⭐⭐⭐☆ (4/5)
  • Action: APPROVE
  • Reasoning: Well-structured release documentation with accurate changelog, justified version bump rationale, and consistent version references. Minor stylistic preference noted but doesn't affect approval. Ready for merge to proceed with GitHub Release creation.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Apr 17, 2026

Greptile Summary

Documentation-only release prep for v3.9.4: adds a CHANGELOG entry covering seven bug fixes, three performance improvements, and one chore; bumps the ROADMAP's current version and date; and adds Phase 8.3 progress notes reflecting the callback-resolution heuristic shipped in #947. No code changes are included.

Confidence Score: 5/5

Safe to merge — documentation-only, no code changes.

All three files are documentation (CHANGELOG, ROADMAP, BACKLOG). Content is internally consistent, PR-linked entries are plausible, dates match, and the patch-level rationale for the two feat: commits is explicitly documented. No P0/P1 findings.

No files require special attention.

Important Files Changed

Filename Overview
CHANGELOG.md New v3.9.4 entry added with accurate PR-linked bug fix, performance, and chore entries matching the described commits
docs/roadmap/ROADMAP.md Current version bumped to 3.9.4, date updated, and Phase 8.3 progress block added with accurate scope notes
docs/roadmap/BACKLOG.md Last-updated date bumped to 2026-04-17; no other changes

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Merge PR #952] --> B[Create GitHub Release\ntag v3.9.4]
    B --> C[publish.yml triggered]
    C --> D[commit-and-tag-version\nbumps package.json]
    D --> E[Native binary builds\nper platform and arch]
    E --> F[npm publish\n@optave/codegraph@3.9.4]
    F --> G[Optional packages published\nper platform and arch]
Loading

Reviews (1): Last reviewed commit: "docs: prepare release notes for v3.9.4" | Re-trigger Greptile

@carlos-alm carlos-alm merged commit 6fa49a8 into main Apr 17, 2026
13 checks passed
@carlos-alm carlos-alm deleted the release/3.9.4 branch April 17, 2026 22:09
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 17, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant