Code review: offsync Documented verify step is not implemented as a command

[ajbeaver]

Generated by Offband issue reporting.

Summary

Code review: offsync Documented verify step is not implemented as a command

Routing

• Repo: offband/offsync
• Repo class: public_tool
• Rule id: code_review.docs_contract
• Candidate key: code-review:offsync:init
• Category: code-review
• Severity: warning
• Review profile: code_review
• Review target: offsync
• Labels: offband-report, weekly, needs-owner, code-review
• Source: code_review.bounded_ai

Expected

Product repositories should avoid actionable code risks identified by bounded source review.

Observed

README.md:11: The README presents the sync workflow as plan -> apply -> verify, but the CLI parser only registers init, link, target, install-deps, plan, and apply; there is no verify subcommand in offsync/cli.py.

Why It Matters

Users following the documented workflow may attempt an unavailable command, and the public contract overstates the command surface for post-apply verification.

Decision Needed

Either document verification as part of apply only, or add a first-class verify command if it is intended to be user-invokable.

Policy Source

config/issue-reporting.json: code_review

Evidence

{
  "artifact_path": "<redacted-path>",
  "canonical_topic": "init",
  "dedupe_core": "init",
  "dedupe_subject": "identifier:init",
  "finding": {
    "confidence": "high",
    "finding_class": "docs_contract",
    "impact": "Users following the documented workflow may attempt an unavailable command, and the public contract overstates the command surface for post-apply verification.",
    "line": 11,
    "owner_decision": "Either document verification as part of `apply` only, or add a first-class `verify` command if it is intended to be user-invokable.",
    "path": "README.md",
    "summary": "The README presents the sync workflow as `plan -> apply -> verify`, but the CLI parser only registers `init`, `link`, `target`, `install-deps`, `plan`, and `apply`; there is no `verify` subcommand in `offsync/cli.py`.",
    "title": "Documented verify step is not implemented as a command"
  },
  "line": 11,
  "path": "README.md",
  "semantic_topic": "init"
}

Suggested Resolution

Either document verification as part of apply only, or add a first-class verify command if it is intended to be user-invokable.

Report

• Report generated: 2026-06-19T10:36:10Z
• Candidate key: code-review:offsync:init