Conversation
|
Claude finished @carlos-alm's task —— View job PR Review Complete ✅Todo List
Phase 1: Root Cause Analysis ✅Problem Being Solved: This is a legitimate release PR implementing Phase 7 (Expanded Language Support) from the roadmap. It adds 6 new languages (C, C++, Kotlin, Swift, Scala, Bash) and includes necessary version bumps and documentation updates. Why This Approach: Standard release process using Risk Assessment: Low risk - this is administrative release preparation, not code changes that could hide issues. Phase 2: Configuration Change Detection ✅No High-Risk Changes Detected:
Phase 3: Test Integrity ✅No Test Weakening Detected:
Phase 4: Code Quality ✅Release Quality Assessment:
Phase 5: Backlog Compliance ✅Root Cause AnalysisProblem Being Solved: Standard release process - version bump to v3.6.0 after completing Phase 7.1 (6 new languages added) Backlog Compliance
Critical Concerns (if any)None identified. This is a clean release PR following established process. Final Recommendation
|
Greptile SummaryThis is a clean release PR bumping the project to v3.6.0, covering documentation updates, version synchronization across all manifests, and a lock file restoration. All three issues flagged in prior review rounds (CHANGELOG language count, multi-language differentiator row, and Phase 7 overview description) have been resolved in commit Key changes:
Confidence Score: 5/5Safe to merge — all previously flagged issues are resolved and no new issues are present. All three P1/P2 issues from prior review rounds are confirmed fixed in 78840a1. The version bump is consistent across all four manifests. The lock file libc restoration is correct for napi-rs Linux platform packages. Documentation is internally consistent across CHANGELOG, README, and ROADMAP. No logic, syntax, or data-integrity issues found. No files require special attention. Important Files Changed
Reviews (2): Last reviewed commit: "fix(docs): correct language count and up..." | Re-trigger Greptile |
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
CHANGELOG.md
Outdated
|
|
||
| ## [3.6.0](https://github.com/optave/ops-codegraph-tool/compare/v3.5.0...v3.6.0) (2026-03-30) | ||
|
|
||
| **Six new languages and a parser abstraction layer.** This release adds first-class support for C, C++, Kotlin, Swift, Scala, and Bash — bringing the total supported languages to 14. A new parser abstraction layer decouples language extractors from tree-sitter internals, making it straightforward to add more languages. The native Rust engine gains batched query methods for the read path, WAL corruption is fixed when native and JS connections overlap, and WASM call-AST extraction is restored for full engine parity. |
There was a problem hiding this comment.
The release summary states bringing the total supported languages to 14, but this is incorrect. The previous total was 11, and 6 new languages are added in this release (C, C++, Kotlin, Swift, Scala, Bash), making the new total 17 — which is reflected consistently everywhere else in this PR (README comparison table, feature descriptions, notes, etc.).
| **Six new languages and a parser abstraction layer.** This release adds first-class support for C, C++, Kotlin, Swift, Scala, and Bash — bringing the total supported languages to 14. A new parser abstraction layer decouples language extractors from tree-sitter internals, making it straightforward to add more languages. The native Rust engine gains batched query methods for the read path, WAL corruption is fixed when native and JS connections overlap, and WASM call-AST extraction is restored for full engine parity. | |
| **Six new languages and a parser abstraction layer.** This release adds first-class support for C, C++, Kotlin, Swift, Scala, and Bash — bringing the total supported languages to 17. A new parser abstraction layer decouples language extractors from tree-sitter internals, making it straightforward to add more languages. The native Rust engine gains batched query methods for the read path, WAL corruption is fixed when native and JS connections overlap, and WASM call-AST extraction is restored for full engine parity. |
There was a problem hiding this comment.
Fixed in 78840a1 — corrected the language count from 14 to 17 in the CHANGELOG summary.
README.md
Outdated
| @@ -106,7 +106,7 @@ No config files, no Docker, no JVM, no API keys, no accounts. Point your agent a | |||
| | **💥** | **Git diff impact** | `codegraph diff-impact` shows changed functions, their callers, and full blast radius — enriched with historically coupled files from git co-change analysis. Ships with a GitHub Actions workflow | | |||
| | **🌐** | **Multi-language, one graph** | JS/TS + Python + Go + Rust + Java + C# + PHP + Ruby + HCL in a single graph — agents don't need per-language tools | | |||
There was a problem hiding this comment.
The differentiator table entry for "Multi-language, one graph" still enumerates only JS/TS + Python + Go + Rust + Java + C# + PHP + Ruby + HCL (the original 11 languages), without mentioning the 6 new languages added in this release. While the total count is correctly updated to 17 in the comparison table and notes, this specific line is now inconsistent and could confuse readers.
| | **🌐** | **Multi-language, one graph** | JS/TS + Python + Go + Rust + Java + C# + PHP + Ruby + HCL in a single graph — agents don't need per-language tools | | |
| | **🌐** | **Multi-language, one graph** | JS/TS + Python + Go + Rust + Java + C# + PHP + Ruby + C + C++ + Kotlin + Swift + Scala + Bash + HCL in a single graph — agents don't need per-language tools | |
There was a problem hiding this comment.
Fixed in 78840a1 — added all 6 new languages (C, C++, Kotlin, Swift, Scala, Bash) to the multi-language differentiator row.
docs/roadmap/ROADMAP.md
Outdated
| | [**6**](#phase-6--native-analysis-acceleration) | Native Analysis Acceleration | Rust extraction for AST/CFG/dataflow/complexity; batch SQLite inserts; incremental rebuilds; native DB write pipeline; full rusqlite migration so native engine never touches better-sqlite3 | **Complete** (v3.5.0) | | ||
| | [**7**](#phase-7--analysis-depth) | Analysis Depth | TypeScript-native resolution, inter-procedural type propagation, field-based points-to analysis, enhanced dynamic dispatch, barrel file resolution, precision/recall CI gates | Planned | | ||
| | [**8**](#phase-8--expanded-language-support) | Expanded Language Support | Parser abstraction layer, 23 new languages in 4 batches (11 → 34), dual-engine support | Planned | | ||
| | [**7**](#phase-7--expanded-language-support) | Expanded Language Support | Parser abstraction layer, 23 new languages in 4 batches (11 → 34), dual-engine support | **In Progress** (v3.6.0) | |
There was a problem hiding this comment.
The overview table still describes Phase 7 with the original plan: 23 new languages in 4 batches (11 → 34). Now that Batch 1 (6 languages) has shipped in v3.6.0, this description should reflect the remaining work. The README roadmap section was correctly updated to say 17 remaining in 3 batches (17 → 34), but the ROADMAP.md overview table was not.
| | [**7**](#phase-7--expanded-language-support) | Expanded Language Support | Parser abstraction layer, 23 new languages in 4 batches (11 → 34), dual-engine support | **In Progress** (v3.6.0) | | |
| | [**7**](#phase-7--expanded-language-support) | Expanded Language Support | Parser abstraction layer, 23 new languages in 4 batches (11 → 34), dual-engine support — Batch 1 (6 languages) shipped in v3.6.0; 17 remaining in 3 batches (17 → 34) | **In Progress** (v3.6.0) | |
There was a problem hiding this comment.
Fixed in 78840a1 — updated the Phase 7 overview table to note Batch 1 shipped in v3.6.0 with 17 remaining in 3 batches.
- CHANGELOG: fix total language count from 14 to 17 - README: add 6 new languages to multi-language differentiator row - ROADMAP: update Phase 7 overview to reflect Batch 1 completion
1 similar comment
1 participant
Summary
libcfields in package-lock.json Linux entriesTest plan
npm installsucceeds with updated lock file