Skip to content

feat: add advisory docs sync review for pull requests#94

Merged
beubax merged 3 commits into
mainfrom
72-feature-add-pr-review-action-for-docs-skills-and-readme-sync
Jul 15, 2026
Merged

feat: add advisory docs sync review for pull requests#94
beubax merged 3 commits into
mainfrom
72-feature-add-pr-review-action-for-docs-skills-and-readme-sync

Conversation

@rishabhraj36

@rishabhraj36 rishabhraj36 commented Jul 14, 2026

Copy link
Copy Markdown
Collaborator

Description

Adds an advisory documentation-sync review for pull requests.

The workflow checks whether user-facing changes are reflected in README.md, docs/, and bundled skills/. Obvious non-documentation changes are resolved deterministically; relevant code and documentation changes receive a Gemini review. Results are advisory and never block merging.

Maintainers can apply the docs-not-needed label when no update is required.

Closes #72

Type of Change

  • ✨ New feature
  • 📝 Documentation
  • 🔧 CI / build / tooling

Checklist

  • I ran the checks relevant to this PR
  • I updated tests or docs if needed
  • I included output or screenshots when useful

Adapter Notes

  • Updated generated or lean docs when command discoverability changed
  • Used positional args for the command's primary subject unless a named flag is clearly better
  • Normalized expected adapter failures to CliError subclasses instead of raw Error

Screenshots / Output

No UI changes. Verification completed with the full test suite and TypeScript typecheck.

@rishabhraj36 rishabhraj36 linked an issue Jul 14, 2026 that may be closed by this pull request
@rishabhraj36 rishabhraj36 marked this pull request as ready for review July 15, 2026 07:49
@rishabhraj36

Copy link
Copy Markdown
Collaborator Author

🟢 No documentation gap found — low confidence

The pull request introduces a new CI workflow for documentation synchronization review and updates `CONTRIBUTING.md` to describe this process. The changes are primarily internal tooling and contributor-facing, and the relevant documentation (`CONTRIBUTING.md`) has been updated. No user-facing behavior of the `webcmd` tool itself has changed, so `README.md`, `docs/`, or `skills/` do not require updates.

Limitations

  • Review context was truncated.

Source: gemini.

This review is advisory and does not block merging.

@beubax beubax merged commit 0fed2ec into main Jul 15, 2026
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature]: Add PR review action for docs, skills, and README sync

2 participants