Skip to content

docs: correct the Whitebox per-category tool counts - #1860

Merged
giswqs merged 1 commit into
mainfrom
fix/whitebox-tool-counts
Aug 12, 2026
Merged

docs: correct the Whitebox per-category tool counts#1860
giswqs merged 1 commit into
mainfrom
fix/whitebox-tool-counts

Conversation

@giswqs

@giswqs giswqs commented Aug 12, 2026

Copy link
Copy Markdown
Member

Summary

  • The README table of Whitebox tools per category listed 280 vector and 232 raster tools; the generated menu catalog has 313 and 256. The nine rows now sum to 1,066, matching the catalog.
  • The matching approximate counts on the features page were updated the same way (vector ~315, raster ~255, remote sensing ~155).

Test plan

  • Counts match the per-category totals in apps/geolibre-desktop/src/lib/whitebox-menu-catalog.ts (1,066 tools across 9 categories).
  • README table and the features page render correctly.

Summary by CodeRabbit

  • Documentation
    • Updated documented geoprocessing tool counts for vector and raster tools.
    • Revised Whitebox toolbox category counts, including vector, raster, and remote-sensing tools.

The vector and raster rows drifted from the generated menu catalog, which
now holds 1,066 tools across the nine categories.
Copilot AI lite review requested due to automatic review settings August 12, 2026 03:54

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@coderabbitai

coderabbitai Bot commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 28b4b594-1643-45f6-9882-59901860effe

📥 Commits

Reviewing files that changed from the base of the PR and between 220e8a4 and a7c57ce.

📒 Files selected for processing (2)
  • README.md
  • docs/features.md

📝 Walkthrough

Walkthrough

The README and feature documentation now show updated vector, raster, and remote-sensing tool counts.

Changes

Geoprocessing tool count documentation

Layer / File(s) Summary
Update documented tool counts
README.md, docs/features.md
Updated the documented vector, raster, and remote-sensing tool counts.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

Poem

A rabbit hops through counts so bright,
Vector and raster rise in sight.
Remote sensing joins the cheer,
Fresh numbers guide the reader clear.
“Nibble well!” says Bun, with delight.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/whitebox-tool-counts

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.

@giswqs
giswqs merged commit 0dc5abc into main Aug 12, 2026
8 of 10 checks passed
@giswqs
giswqs deleted the fix/whitebox-tool-counts branch August 12, 2026 03:56
@github-actions

Copy link
Copy Markdown
Contributor

I manually re-derived every category total from the checked-in apps/geolibre-desktop/src/lib/whitebox-menu-catalog.ts (counting { id: ... } tool entries per top-level category) and cross-checked against this PR's new numbers: Vector = 313 (exact match), Raster = 256 (exact match), Hydrology = 100 (unchanged row, exact match), Remote sensing = 154 (unchanged row, exact match), and the nine-category total = 1,066, matching the file's own header comment (/** 1066 tools across 9 categories. */). The docs/features.md approximations (~315, ~255, ~155) also follow the file's existing nearest-5 rounding convention. No factual, formatting, or style problems found.

No inline comments were posted — the diff is accurate.

Code review

No issues found. This is a docs-only correction (README.md and docs/features.md) updating stale Whitebox tool-count figures. I independently recounted every tool entry in the auto-generated whitebox-menu-catalog.ts for Vector, Raster, Hydrology, and Remote sensing categories, and all match the PR's new numbers exactly; the nine-category sum (1,066) also matches the catalog file's own header comment. The docs/features.md approximate figures follow the existing nearest-5 rounding convention used elsewhere in that line.

  • Bugs: None found.
  • Security: None found — no code paths touched, docs only.
  • Performance: Not applicable.
  • Quality: None found — wording and formatting consistent with surrounding table/list style.
  • CLAUDE.md: No violations — this PR only corrects documentation numbers and does not touch whitebox-menu-catalog.ts itself, so the "regenerate with scripts/gen-whitebox-menu-catalog.mjs" rule doesn't apply here.

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