Skip to content

[codex] Add deslopify review mode#55

Closed
mbelinky wants to merge 1 commit into
openclaw:mainfrom
mbelinky:codex/deslopify-review-mode
Closed

[codex] Add deslopify review mode#55
mbelinky wants to merge 1 commit into
openclaw:mainfrom
mbelinky:codex/deslopify-review-mode

Conversation

@mbelinky
Copy link
Copy Markdown
Contributor

Summary

Adds a narrow review --mode deslopify lane for slop-driven simplification and performance findings. The default review path remains broad for correctness, security, API contract, and other risk categories.

The new mode:

  • passes deslopify-specific instructions into the review prompt
  • restricts stored provider findings to maintainability and performance
  • rejects unsupported review modes such as simplify
  • documents the narrow rubric and CLI usage

Validation

  • pnpm exec oxfmt --write README.md docs/code-review.md src/app.ts src/cli.ts src/prompt.ts src/workflow.test.ts
  • pnpm test src/workflow.test.ts
  • pnpm typecheck
  • pnpm build
  • git diff --check
  • local CLI smoke: review --mode simplify --dry-run rejects the mode
  • local CLI smoke: review --mode deslopify --limit 1 --dry-run reports mode: deslopify

Notes

This intentionally does not add report clustering or deslopify-specific ranking; that should be a follow-up PR.

@steipete
Copy link
Copy Markdown
Contributor

This stacked change landed through #56 in eabfff0. Closing this lower PR to keep the queue clean.

Thanks @mbelinky.

@steipete steipete closed this May 18, 2026
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