Skip to content

bug: plan verify hint omits --repo and hard-codes main...HEAD #110

Description

@aryamthecodebreaker

What happened?

After plan --format json --output <file>, stderr always suggests:

fixmap verify --report <file> --diff main...HEAD

Problems:

  1. --repo is omitted even when the plan scanned a remote GitHub URL or an auto-resolved issue repository (temp clone). Running that verify hint from the user’s cwd verifies the wrong tree (or fails).
  2. main...HEAD is hard-coded when the user did not pass --diff — many repos use master, and the user’s work may be on another base.

Observed while planning https://github.com/chalk/chalk/issues/1 with --output from a FixMap working directory: verify hint pointed at main...HEAD with no chalk --repo.

Source: nextCommandHint in packages/cli/src/cli-runner.ts (options.diffSpec ?? "main...HEAD", no repo flag).

How can we reproduce it?

cd /path/to/FixMap
npx -y @aryam/fixmap@0.7.3 plan \
  --issue https://github.com/chalk/chalk/issues/1 \
  --format json --output chalk-plan.json
# read stderr verify hint — no --repo, assumes main...HEAD

What did you expect instead?

Hint should include the repo that was actually scanned (--repo <path-or-url>) when it is not cwd, and should not invent main...HEAD unless that ref exists / was used. If verify cannot run against a temp clone, say so instead of emitting a broken command.

FixMap version or commit

v0.7.3 / main

Where did this happen?

CLI

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't workinggood first issueApproachable contribution with a clear scope

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions