Add Improvements.md with future directions #32

New issue

Closed

opened 2026-06-27 07:13:32 +00:00 by coding-agent-marvin8 · 1 comment

coding-agent-marvin8 commented

2026-06-27 07:13:32 +00:00

(Migrated from codeberg.org)

WuMing has reached feature-complete status on its initial roadmap. This issue tracks creating an Improvements.md (Yilan-format grouped checklist) at the repo root, capturing the full set of potential future directions identified from a code review: CI platform expansion (Forgejo Actions, GitHub Actions), backend and agent expansion, review quality improvements, configuration/UX additions, reliability gaps, observability, and build/deployment hardening. A sidecar document will cover the Forgejo Actions integration design in detail.

coding-agent-marvin8 commented

2026-06-27 07:13:45 +00:00

(Migrated from codeberg.org)

Code review findings and plan of attack:

Codebase state: All planned phases implemented. Clean architecture — Backend protocol, BaseAgent/Router pattern, async throughout, sha256 dedup, three backends (DeepSeek/Anthropic/Ollama), five agents (code/config/docs/security/shell), structured JSON logging, retry with exponential backoff, rich progress display.

Notable gaps found:

Deduplication hash covers only path:line but concept.md says it should cover diff-hunk content — spec and implementation are out of sync
list_review_comment_bodies fetches all reviews in a single un-paginated call
No config validation at startup (empty token, missing backend key silently proceeds)
Backend HTTP timeout hardcoded to 120 s with no override

Plan: Create two files in the repo root:

Improvements.md — Yilan-format grouped checklist (CI Platform Support, Backend Expansion, Agent Expansion, Review Quality, Configuration & UX, Reliability, Observability, Build & Deployment)
Improvements-forgejo-actions.md — sidecar design notes for the Forgejo Actions integration (action.yml skeleton, env var mapping table, PR number extraction from GITHUB_EVENT_PATH JSON, config.py dual-platform support)

Branch: feat/issue-32-improvements-md

Code review findings and plan of attack: **Codebase state:** All planned phases implemented. Clean architecture — Backend protocol, BaseAgent/Router pattern, async throughout, sha256 dedup, three backends (DeepSeek/Anthropic/Ollama), five agents (code/config/docs/security/shell), structured JSON logging, retry with exponential backoff, rich progress display. **Notable gaps found:** - Deduplication hash covers only `path:line` but concept.md says it should cover diff-hunk content — spec and implementation are out of sync - `list_review_comment_bodies` fetches all reviews in a single un-paginated call - No config validation at startup (empty token, missing backend key silently proceeds) - Backend HTTP timeout hardcoded to 120 s with no override **Plan:** Create two files in the repo root: 1. `Improvements.md` — Yilan-format grouped checklist (CI Platform Support, Backend Expansion, Agent Expansion, Review Quality, Configuration & UX, Reliability, Observability, Build & Deployment) 2. `Improvements-forgejo-actions.md` — sidecar design notes for the Forgejo Actions integration (action.yml skeleton, env var mapping table, PR number extraction from GITHUB_EVENT_PATH JSON, config.py dual-platform support) Branch: `feat/issue-32-improvements-md`

No labels

No milestone

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Reference

marvin8/wuming#32

No description provided.

Rows
Columns