Start flow steps with a divergence check against origin/main

Open
#138 0 comments 0 reactions 0 assignees View on GitHub

Maintainers usually reply within 1 day

Nobody has claimed this yet.

Assessment

Difficulty
3/5
Estimated time
Half a day
Newbie friendliness
45/100
Issue type
Feature
Clarity
Mostly clear
Activity status
Active
Tech stack
git, github, typescript

Research direction

The flow-reminders hook and the workflow doc are the two places to change; find where the hook injects reminders for /implement, /implement-spec and /pr, and where each step's guard rails are written. Add the git fetch and git log HEAD..origin/main check to all three steps, and make a step that finds new commits report them and ask before merging. Done when the hook's tests cover the new reminder and the workflow doc names the check in each step.

Written by the indexing model from the issue text.

Description

ready-for-agent

What to build

/implement, /implement-spec and /pr begin with git fetch and git log HEAD..origin/main, and report any new commits before continuing. The check runs through the flow-reminders hook and a step in the workflow doc, so it does not depend on an agent remembering.

Origin: the retro of PR #134. main moved mid-session (an oxc and Biome fix) and the maintainer had to tell the agent to pull it in.

Acceptance criteria

  • The hook injects the divergence check for the three flow steps.
  • The workflow doc names the check in each step's guard rail.
  • A step that finds new commits on main reports them and asks before merging them in.
  • The hook's tests cover the new reminder.

Blocked by

None (can start immediately).

Dominant language
TypeScript
Stars
1
Forks
0
Avg merge
2h 8m
Merged PRs (30d)
64

Getting set up

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from nemchik/biome-plugin-suitescript

All issues in nemchik/biome-plugin-suitescript

Similar issues

More TypeScript issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.