Skip to content

docs: remove stale planning notes#9342

Merged
rubenfiszel merged 1 commit into
mainfrom
clean-stale-docs
May 27, 2026
Merged

docs: remove stale planning notes#9342
rubenfiszel merged 1 commit into
mainfrom
clean-stale-docs

Conversation

@centdix
Copy link
Copy Markdown
Collaborator

@centdix centdix commented May 27, 2026

Summary

Clean up stale docs by removing historical planning/status notes that no longer match the current repository state, while keeping the active app-mode recommendation note concise.

Changes

  • Trim docs/app-mode-ai-chat-review.md down to core next-step recommendations for improving app-mode AI chat.
  • Remove obsolete app-token baseline, failing-test tracker, and system-prompt testing plan/status docs.
  • Keep the retained AI refactor and GitRepoViewer hub docs unchanged.

Test plan

  • git diff --check main...HEAD
  • git grep confirms no remaining references to removed docs

@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying windmill with  Cloudflare Pages  Cloudflare Pages

Latest commit: 93d88ce
Status: ✅  Deploy successful!
Preview URL: https://d4f46da0.windmill.pages.dev
Branch Preview URL: https://clean-stale-docs.windmill.pages.dev

View logs

@centdix centdix marked this pull request as ready for review May 27, 2026 02:45
@claude
Copy link
Copy Markdown
Contributor

claude Bot commented May 27, 2026

Claude finished @centdix's task in 53s —— View job


Verdict

Good to merge — pure docs cleanup; no code paths affected and no remaining references to the removed files.

What I checked

  • Diff vs origin/main: only docs/ files; 4 deletions + 1 trim of docs/app-mode-ai-chat-review.md down to a 49-line "Recommended Next Steps" list. Reads coherently on its own.
  • git grep for the four deleted filenames (app-mode-ai-chat-token-baseline, failing-tests.md, system-prompt-testing-plan, system-prompt-testing-status): zero hits, so nothing links into the removed pages.
  • git grep for app-mode-ai-chat-review: also zero hits — the retained file is a freestanding note, not referenced elsewhere, which matches its tone ("only tracks the highest-value next steps").
  • docs/ directory listing after the change: clean (autonomous-mode.md, enterprise.md, fork-triggers.md, git-repo-viewer-hub-script.md, validation.md, windmill-ai-refactor-plan.md, plus the trimmed review and a couple of helper assets). The AGENTS.md references to docs/validation.md and docs/enterprise.md still resolve.

Findings

None.

Test coverage

Docs-only change — no automated tests expected and none needed. No manual verification required; the Cloudflare Pages preview build already passed.

Todo

  • Inspect diff
  • Verify no remaining references to deleted docs
  • Check retained file is coherent
  • Post review
    clean-stale-docs

Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

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

No issues found across 5 files

Re-trigger cubic

@github-actions
Copy link
Copy Markdown
Contributor

Pi Review

Good to merge

This PR removes 1,755 lines of stale documentation across five files — historical token baselines, a failing-test tracker, an outdated system-prompt testing plan, and a status doc — while trimming docs/app-mode-ai-chat-review.md to seven concise next-step recommendations.

No issues to flag:

  • No code changes. Every file touched is a .md under docs/.
  • No remaining cross-references. A grep for the deleted filenames found no hits in any project files (only in pi-events.jsonl — this review's own tool-execution log).
  • No new public surfaces to review for auth, validation, or module-placement correctness.
  • No AGENTS.md violations. The retained docs/app-mode-ai-chat-review.md is a clean list of recommendations with no code to evaluate.

Test coverage

CI / workflows / docs / config-only — no automated tests expected. The PR author's own test plan (git diff --check, git grep) covers the verifyable properties of a doc-deletion change. No manual verification is needed — there is no in-app surface to exercise.

@github-actions
Copy link
Copy Markdown
Contributor

Codex Review

Good to merge.

No issues found. Checked for bugs, security, AGENTS.md compliance, and broken references to the deleted docs.

Test coverage

No automated tests are expected for this docs-only diff. Manual verification is limited to confirming the removed planning/status files are not referenced elsewhere; there is no in-app surface to exercise.

@rubenfiszel rubenfiszel merged commit 9752f0f into main May 27, 2026
27 checks passed
@rubenfiszel rubenfiszel deleted the clean-stale-docs branch May 27, 2026 07:19
@github-actions github-actions Bot locked and limited conversation to collaborators May 27, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants