Repository navigation
Conversation
clawgenti
force-pushed
the
update/doc-drift
branch
from
November 5, 2026 17:00
9eef3a4 to
365002f
Compare
clawgenti
changed the title
jimcadden
added
the
ready-for-ai-review
jimcadden
approved these changes
Nov 5, 2026
jimcadden
left a comment
jimcadden
left a comment
Contributor
There was a problem hiding this comment.
Both changes check out against the tree; two wording/style nits inline, one stale line in the same section.
Verified
set-requirementsis implemented (cmd_set_requirements,repoman-setup.sh:483) and was already advertised inusage()(:60); the file header was the only place missing it, so the comment change now matches the script.- Both citations are complete and correct. Exactly two PRs merged between
v0.1.0(2026-10-27T19:04Z) andv0.1.1(2026-11-05T15:51Z): #9 (banner) and #12 (doc-drift) — so the[0.1.1]"Added" list has no gaps. The banner really is new in 0.1.1 (## Startup bannerexists inskills/repoman/SKILL.mdonv0.1.1, absent onv0.1.0). - Composes cleanly with #14 and #15: applying 14 → 15 → 16 to a clean
8a679b0gives no conflicts.
Author: clawgenti (COLLABORATOR — maintainer)
Areas reviewed: Docs (CHANGELOG), Shell (comment only), CI
Agent/IDE config (.claude/.vscode): none
Commits: 1, signed-off: yes
CI status: 4/4 passing at 365002f
Non-blocking notes:
CHANGELOG.md:17(in the section this PR edits, though outside the diff) still says "Thev0.1.1tag is cut frommainafter this release merges." That is now history —v0.1.1was published2026-11-05T15:52Z. Since you are already touching this section, worth rewording to past tense or dropping the sentence.- Commit subject is a bare bullet list with no conventional prefix (
docs: Add set-requirements to the setup header …), advisory per the repo's commit conventions; the PR title is fine. - PR-body
Generated by …footer →Assisted-By: Claude Codeper CLAUDE.md § PR Description Attribution (template fix belongs upstream in the doc-drift fixer).
Verdict: APPROVE.
Generated by RepoMan github-pr-review@235a7f7 (https://github.com/rossoctl/agent-skills/blob/235a7f7620139ee3e4ab0872b8439591fe52e8c1/skills/github-pr-review/SKILL.md).
|
|
||
| - **doc-drift program.** The scanner and fixer compare documentation to the code at a pinned git ref (paths, flags, enumerations, retired names, versions, and absence claims) and draft Keep a Changelog notes from merged pull requests. It ships inside the `repoman` plugin and as its own marketplace plugin. | ||
| - **doc-drift program.** The scanner and fixer compare documentation to the code at a pinned git ref (paths, flags, enumerations, retired names, versions, and absence claims) and draft Keep a Changelog notes from merged pull requests. It ships inside the `repoman` plugin and as its own marketplace plugin. (#12) | ||
| - **REPOMAN banner** on interactive sessions. (#9) |
Contributor
There was a problem hiding this comment.
nit: the banner is narrower than "interactive sessions" — skills/repoman/SKILL.md § Startup banner says to print it "On a human /repoman turn" and to skip it on schedule/cron, headless -p, pipeline and CI runs. Suggest: - **REPOMAN banner** on human /repoman turns. (#9)
| ### Added | ||
|
|
||
| - **doc-drift program.** The scanner and fixer compare documentation to the code at a pinned git ref (paths, flags, enumerations, retired names, versions, and absence claims) and draft Keep a Changelog notes from merged pull requests. It ships inside the `repoman` plugin and as its own marketplace plugin. | ||
| - **doc-drift program.** The scanner and fixer compare documentation to the code at a pinned git ref (paths, flags, enumerations, retired names, versions, and absence claims) and draft Keep a Changelog notes from merged pull requests. It ships inside the `repoman` plugin and as its own marketplace plugin. (#12) |
Contributor
There was a problem hiding this comment.
suggestion: these are the first PR citations in this file — every other bullet, including the Shellcheck one two lines below, has none, so the section becomes inconsistent. If the intent is traceability, cite every entry and link it Keep-a-Changelog style (([#12](https://github.com/rossoctl/repoman/pull/12))); a bare #12 is plain text in Markdown, not a link. Otherwise drop both cites and keep the body uniform.
- Document set-requirements in repoman-setup.sh header subcommand list. - Cite merged PRs rossoctl#12 and rossoctl#9 in the [0.1.1] changelog. Automated update by RepoMan Doc-Drift Fixer (2026-11-05). Signed-off-by: Clawgenti <clawgenti@users.noreply.github.com>
clawgenti
force-pushed
the
update/doc-drift
branch
from
November 9, 2026 16:37
72c659b to
32a0cef
Compare
Collaborator
Author
|
Superseded by #22, which combines this change with the other doc-drift fixes. |
jimcadden
pushed a commit
that referenced
this pull request
Nov 9, 2026
* docs: fix release 0.1.1 date is 2026-11-05 Automated fix by RepoMan Doc-Drift Fixer (2026-11-05). Signed-off-by: 65532 user <65532@agent-f6a74c71745da212-0.agent-f6a74c71745da212.dam-prod-agents.svc.cluster.local> (cherry picked from commit 6c0b167) * docs: fix release 0.1.0 date is 2026-10-27 Automated fix by RepoMan Doc-Drift Fixer (2026-11-05). Signed-off-by: 65532 user <65532@agent-f6a74c71745da212-0.agent-f6a74c71745da212.dam-prod-agents.svc.cluster.local> (cherry picked from commit 6b7dc3f) * docs: doc-drift updates - Document set-requirements in repoman-setup.sh header subcommand list. - Cite merged PRs #12 and #9 in the [0.1.1] changelog. Automated update by RepoMan Doc-Drift Fixer (2026-11-05). Signed-off-by: Clawgenti <clawgenti@users.noreply.github.com> (cherry picked from commit 32a0cef) --------- Signed-off-by: 65532 user <65532@agent-f6a74c71745da212-0.agent-f6a74c71745da212.dam-prod-agents.svc.cluster.local> Signed-off-by: Clawgenti <clawgenti@users.noreply.github.com> Co-authored-by: Clawgenti <clawgenti@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Rolling documentation updates from RepoMan doc-drift.
Changes
skills/repoman/scripts/repoman-setup.sh— add theset-requirementssubcommand to the header's subcommand list (it was already implemented and documented inusage(), but missing from the header summary).CHANGELOG.md— cite merged PRs feat: Add doc-drift program (scanner, fixer, release assist) #12 (doc-drift program) and feat: Show a REPOMAN banner on interactive sessions #9 (REPOMAN banner) in the existing[0.1.1]→### Addedsection.Note
This fixes entries in the existing
[0.1.1]changelog section. It does not publish or edit a GitHub release.Generated by RepoMan doc-drift-fixer (https://github.com/rossoctl/agent-skills/blob/main/skills/doc-drift-fixer/SKILL.md).