Skip to content

feat(web): Svelte web UI + name-storage refactor, branch labels, profiler (v2.2.7) - #43

Merged
hungpham10 merged 3 commits into
mainfrom
feat/web-ui
Oct 7, 2026
Merged

hungpham10 merged 3 commits into
mainfrom
feat/web-ui

Conversation

@hungpham10

Copy link
Copy Markdown
Owner

Tóm tắt

  • Web UI (Svelte): crate codegraph-web nhúng SPA qua rust-embed (allow_missing), phục vụ làm fallback của server GraphQL. Asset build bởi CI/release, không commit.
  • Name storage: persist name engine vào dataset riêng (names.lmdb/names.sqlite) + version stamp → mở lại index bỏ qua rebuild trie; gộp name_records + sorted_name_keys thành một Vec.
  • Branch labels trong flow(); profiler stage (CGPROF).
  • Source: DiskGit trait cho diff/origin simulate.
  • CI/Release: build web asset rồi wire vào tất cả job Rust (clippy/test/slim, deb/rpm, Homebrew, AUR source).
  • Bump version 2.2.7.

Kiểm thử

  • cargo clippy --workspace --all-targets -- -D warnings ✅
  • cargo test -p codegraph-graph --all-features → 124 pass (4 Redis cần server) ✅
  • codegraph-web compile + test với folder assets rỗng/thiếu ✅

…iler

- codegraph-web: embed Svelte SPA (rust-embed, allow_missing) served as
  GraphQL fallback; assets built by CI/release, not committed
- web: SvelteKit SPA (explore/review/documents) + graphql client
- graph: persist name engine in a sibling dataset (names.lmdb/sqlite)
  with version stamp; collapse name_records/sorted_name_keys into one Vec
- graph: branch labels in flow, stage profiler (CGPROF)
- source: DiskGit trait; api/mcp dispatch diff_simulate/origin_simulate
- ci/release: build web assets and wire them into all Rust build jobs;
  deb/rpm, Homebrew, AUR source builds build web first
- bump version to 2.2.7
@coderabbitai

coderabbitai Bot commented Oct 7, 2026 •

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 5f3e4d1a-cba7-494e-b811-48e196c1f73a
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

hungpham10 and others added 2 commits October 7, 2026 06:47
Path Windows dùng '\\' nên name_dsn tạo 'names.sqlite' ở cwd thay vì cạnh
db.sqlite → name_engine_stamp_survives_reopen fail trên CI Windows. Xử lý cả
'/' lẫn '\\' + thêm unit test hồi quy.
@codspeed

codspeed Bot commented Oct 7, 2026 •

Copy link
Copy Markdown

Merging this PR will regress 4 benchmarks

⚡ 4 improved benchmarks
❌ 4 regressed benchmarks
✅ 30 untouched benchmarks

Warning

Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

Benchmark BASE HEAD Efficiency
❌ open+ingest 2 s 2.7 s -24.98%
❌ open+ingest 540.4 ms 662.5 ms -18.42%
❌ open+ingest 1.8 s 2.2 s -16.65%
❌ open+ingest 213.5 ms 251.9 ms -15.24%
⚡ cold_depth1 549.3 ms 365.9 ms +50.13%
⚡ cold_depth1 87.5 ms 60 ms +45.7%
⚡ warm_no_hit 545.8 µs 457.5 µs +19.3%
⚡ open+ingest 137.2 ms 122.5 ms +12.04%

Tip

Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.


Comparing feat/web-ui (6bb556c) with main (8582aeb)

Open in CodSpeed

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 71.35193% with 267 lines in your changes missing coverage. Please review.
✅ Project coverage is 77.93%. Comparing base (8582aeb) to head (6bb556c).

Files with missing lines Patch % Lines
crates/codegraph-api/src/tools.rs 0.00% 73 Missing ⚠️
crates/codegraph-graphql/src/types.rs 0.00% 51 Missing ⚠️
crates/codegraph-source/src/git.rs 60.60% 26 Missing ⚠️
crates/codegraph-graphql/src/lib.rs 74.48% 25 Missing ⚠️
crates/codegraph-core/src/prof.rs 66.15% 22 Missing ⚠️
crates/codegraph-graph/src/lib.rs 88.40% 16 Missing ⚠️
crates/codegraph-mcp/src/lib.rs 0.00% 11 Missing ⚠️
crates/codegraph-web/src/lib.rs 84.72% 11 Missing ⚠️
crates/codegraph-graphql/src/query.rs 55.55% 8 Missing ⚠️
crates/codegraph/src/main.rs 0.00% 8 Missing ⚠️
... and 4 more

❌ Your patch status has failed because the patch coverage (71.35%) is below the target coverage (100.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #43      +/-   ##
==========================================
- Coverage   78.01%   77.93%   -0.09%     
==========================================
  Files          94       96       +2     
  Lines       22348    23105     +757     
==========================================
+ Hits        17435    18007     +572     
- Misses       4913     5098     +185     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@hungpham10
hungpham10 merged commit 8b7dd87 into main Oct 7, 2026
21 of 23 checks passed
@hungpham10
hungpham10 deleted the feat/web-ui branch October 7, 2026 07:55
hungpham10 added a commit that referenced this pull request Oct 8, 2026
…iler (v2.2.7) (#43)

* feat(web): Svelte web UI + name-storage refactor, branch labels, profiler

- codegraph-web: embed Svelte SPA (rust-embed, allow_missing) served as
  GraphQL fallback; assets built by CI/release, not committed
- web: SvelteKit SPA (explore/review/documents) + graphql client
- graph: persist name engine in a sibling dataset (names.lmdb/sqlite)
  with version stamp; collapse name_records/sorted_name_keys into one Vec
- graph: branch labels in flow, stage profiler (CGPROF)
- source: DiskGit trait; api/mcp dispatch diff_simulate/origin_simulate
- ci/release: build web assets and wire them into all Rust build jobs;
  deb/rpm, Homebrew, AUR source builds build web first
- bump version to 2.2.7

* style: apply rustfmt

* fix(graph): sibling_file giữ thư mục cha trên Windows

Path Windows dùng '\\' nên name_dsn tạo 'names.sqlite' ở cwd thay vì cạnh
db.sqlite → name_engine_stamp_survives_reopen fail trên CI Windows. Xử lý cả
'/' lẫn '\\' + thêm unit test hồi quy.

---------

Co-authored-by: Hung Pham <hungpham@MacBook-Pro-cua-Hung.local>
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.

2 participants