Skip to content

fix command bypasses the workflow layer and silently ignores --mode #115

@mayashavin

Description

@mayashavin

Problem

src/cli/commands/fix.ts violates the project's CLI architecture rule: "CLI commands must not contain business logic."

  1. It directly calls parseMarkdownTool, reviewSeoTool, and updateFrontmatterTool instead of delegating to a workflow (lines 25–33).
  2. It accepts --mode light|medium|strong (line 16) but never passes it anywhere — the writing review (reviewWritingTool) is never called, so fix only surfaces SEO critical suggestions, never grammar/writing fixes.

Expected behaviour

  • A fixDraftWorkflow should be created under src/workflows/fixDraftWorkflow.ts that composes parseMarkdownTool, reviewSeoTool, reviewWritingTool, and updateFrontmatterTool.
  • The fix command should delegate to that workflow and pass --mode correctly.
  • Writing suggestions marked with high confidence should be included in the diff output.

Files affected

  • src/cli/commands/fix.ts
  • src/workflows/ (new file needed)

References

  • Architecture rule: CLAUDE.md — "CLI commands must not contain business logic."
  • Plan §7.4 — fix command should show grammar/writing fixes.

Metadata

Metadata

Assignees

No one assigned

    Labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions