Skip to content

Add codebase scanning tool - #437

Open
FFizzZZ wants to merge 3 commits into
sashiko-dev:mainfrom
FFizzZZ:codebase-scan
Open

Add codebase scanning tool#437
FFizzZZ wants to merge 3 commits into
sashiko-dev:mainfrom
FFizzZZ:codebase-scan

Conversation

@FFizzZZ

@FFizzZZ FFizzZZ commented Aug 17, 2026

Copy link
Copy Markdown

Summary

  • add TraeCLI as a review provider and expose structured review findings
  • add a tools/codebase-scan package for whole-codebase scanning with inventory, grouping, synthetic patch generation, Sashiko orchestration, validation, and report rendering
  • wire the tool into project docs, packaging, Makefile tests, and GitHub Actions

Testing

  • Not run in this PR creation step

FFizzZZ and others added 3 commits August 16, 2026 19:03
Add an isolated TraeCLI completion backend and route review
workers through the existing stdio protocol.

Signed-off-by: Jiahao Luo <luojiahao.fs@bytedance.com>
Co-authored-by: TRAE CLI <noreply@bytedance.com>
Let embedding callers consume verified Stage 10 findings without
running the Stage 11 report formatter. Preserve optional repair
guidance.

Signed-off-by: Jiahao Luo <luojiahao.fs@bytedance.com>
Co-authored-by: TRAE CLI <noreply@bytedance.com>
Extend Sashiko beyond per-commit review with an optional Codebase
Scan tool. It inventories existing source trees, reviews deterministic
synthetic diffs, and emits versioned artifacts.

Signed-off-by: Jiahao Luo <luojiahao.fs@bytedance.com>
Co-authored-by: TRAE CLI <noreply@bytedance.com>
@rgushchin

Copy link
Copy Markdown
Member

Thank you for the pull request! I really like the idea of adding a scanning tool, but I think it should be better integrated into the sashiko source code rather than a python wrapper on top of it. Wdyt about such approach?

@FFizzZZ

FFizzZZ commented Aug 18, 2026

Copy link
Copy Markdown
Author

Thank you for the feedback! I'll rework the implementation to integrate the scanning tool directly into the Sashiko source code.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants