Skip to content

fix: consolidate Claude workflows and clarify FOUNDATION.md#24

Merged
carlos-alm merged 2 commits intomainfrom
fix/sync-v2-and-publish-workflow
Feb 23, 2026
Merged

fix: consolidate Claude workflows and clarify FOUNDATION.md#24
carlos-alm merged 2 commits intomainfrom
fix/sync-v2-and-publish-workflow

Conversation

@carlos-alm
Copy link
Contributor

Summary

  • Merge claude-code-review.yml into claude.yml as a dedicated review job, triggered on pull_request events — removes the duplicate standalone workflow
  • Rewrite the "what we're not" section in FOUNDATION.md to better articulate codegraph's positioning as a code intelligence engine

Test plan

  • Verify claude.yml workflow triggers correctly on PR events (review job) and issue/comment events (claude job)
  • Confirm claude-code-review.yml is deleted and no CI references remain

Merge claude-code-review.yml into claude.yml so both on-demand
@claude mentions and automatic PR reviews run from one workflow.
The review job gates on pull_request events while the claude job
gates on @claude mentions — neither will skip unexpectedly.
Replace abstract negations (not a database, not a RAG system) with
concrete product-level distinctions (not an app, not a coding tool,
not an agent) that better communicate codegraph's positioning.
@greptile-apps
Copy link
Contributor

greptile-apps bot commented Feb 23, 2026

Greptile Summary

Consolidated duplicate Claude workflows into a single file and improved documentation clarity. The standalone claude-code-review.yml workflow was merged into claude.yml as a dedicated review job, eliminating duplication while preserving identical functionality. The FOUNDATION.md "what we're not" section was rewritten to better articulate codegraph's positioning as a code intelligence engine rather than focusing on technical implementation details.

Confidence Score: 5/5

  • This PR is safe to merge with no risk
  • Both changes are straightforward refactoring: the workflow consolidation is a mechanical merge with no functional changes, and the documentation update improves clarity without affecting code
  • No files require special attention

Important Files Changed

Filename Overview
.github/workflows/claude.yml Consolidated claude-code-review.yml into this file as a dedicated review job; properly isolated with if condition and identical configuration
.github/workflows/claude-code-review.yml Deleted file — functionality merged into claude.yml as the review job
FOUNDATION.md Improved positioning clarity in "what we're not" section; shifted from technical implementation details to clearer product positioning

Last reviewed commit: 6080e1c

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

3 files reviewed, no comments

Edit Code Review Agent Settings | Greptile

@carlos-alm carlos-alm merged commit 6caf6ce into main Feb 23, 2026
15 of 16 checks passed
@carlos-alm carlos-alm deleted the fix/sync-v2-and-publish-workflow branch February 23, 2026 00:31
carlos-alm added a commit that referenced this pull request Mar 22, 2026
autodev-codebase (3.3) was ranked #23 above Claude-code-memory (3.4)
at #24. Swapped to maintain descending score order.
carlos-alm added a commit that referenced this pull request Mar 23, 2026
…#559)

* docs: update competitive analysis for v3.2.0 and March 2026 landscape

Re-rank codegraph from #8 (4.0) to #5 (4.5) reflecting v3.2.0 features:
41 CLI commands, 32 MCP tools, dataflow across all 11 languages, CFG,
sequence diagrams, architecture boundaries, unified graph model.

Add new competitors: GitNexus (#1, 18k stars), DeusData/codebase-memory-mcp
(#6, 793 stars in 25 days). Update star counts and feature status across
all 85+ ranked projects. Mark 7 roadmap items as DONE. Flag stagnant
projects. Update joern.md (3,021 stars, 75 contributors, 4 community MCP
wrappers) and narsil-mcp.md (129 stars, SPA frontend, +36 security rules,
development paused since Feb 25).

* docs: fix narsil SPA version attribution in competitive analysis overview

Line 18 incorrectly stated "v1.6.1" as the version when the SPA feature
was introduced. The SPA frontend was added in v1.6.0; v1.6.1 is the
current release. Updated to "added v1.6.0, current v1.6.1" to match the
detailed narsil-mcp.md entry.

* docs: remove hardcoded star count from joern comparison table

The "32 stars, growing" value in the Community & maturity row hardcodes
a stale star count. Other comparison tables use "Growing" consistently
for codegraph's community status. Updated to match.

* fix: correct GitNexus score, Tier 2 rank numbering, and jelly star count

- GitNexus overall score corrected from 4.7 to 4.5 to match the
  arithmetic mean of its six sub-scores (5+5+4+4+4+5)/6 = 4.5
- Tier 2 renumbered starting at #38 (was duplicating #37 with Tier 1);
  also resolves the pre-existing duplicate #43 (Bikach/ChrisRoyse now
  #44/#45), with all subsequent entries incremented accordingly
- jelly section header updated from 417 to 423 stars to match the
  ranking table

* fix: correct aider rank and codegraph star count per review feedback

* fix: align scoring breakdown sub-scores with overall rankings for stagnant projects

glimpse: Community 4→2 (stagnant since Jan 2026), avg now 3.83≈3.8 matching ranking.
autodev-codebase: Community 3→1 (stagnant since Jan 2026), avg now 3.33, ranking updated 3.4→3.3.

* fix: align ranking scores with sub-score averages for colbymchenry and axon

* fix: correct ranking inversion at positions #23/#24 (#559)

autodev-codebase (3.3) was ranked #23 above Claude-code-memory (3.4)
at #24. Swapped to maintain descending score order.

* fix: correct score mismatches for code-graph-rag (4.5→4.2) and arbor (3.7→4.2) (#559)

* fix: sync breakdown table row order with ranking table for #23/#24 (#559)

* fix: correct ranking inversions and stale rank references (#559)

* fix: correct sub-score/overall-score mismatches for codexray, loregrep, MATE

* fix: correct score mismatches and aider header rank

* fix: update narsil-mcp Key Metrics to reflect development stagnation (#559)

* fix: add missing "vs arbor" comparison section (#559)

* fix: remove duplicate vs-glimpse section and correct role names in vs-arbor (#559)

The duplicate vs-glimpse block (stale rank #10) was left behind when
vs-arbor was inserted. Removed it — the correct version exists at #11.
Also fixed role vocabulary in vs-arbor: bridge → adapter, added entry.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant