Skip to content

Add doc automation workflows and enhance pr-review#56

Closed
gontzess wants to merge 4 commits intomainfrom
gontzess/automate-connector-docs
Closed

Add doc automation workflows and enhance pr-review#56
gontzess wants to merge 4 commits intomainfrom
gontzess/automate-connector-docs

Conversation

@gontzess
Copy link
Copy Markdown
Contributor

Summary

Adds infrastructure for automated connector documentation as part of the automate-connector-docs project.

  • publish-docs.yaml: Scaffolded reusable workflow for publishing docs/connector.mdx to the connector registry on release. Blocked by registry deployment — active logic is behind TODOs.
  • sync-stable-docs.yaml: Scaffolded reusable workflow for syncing stable channel docs from the registry to ConductorOne/docs. Blocked by registry deployment.
  • release.yaml: Commented publish-docs job block, ready to activate when registry is deployed.
  • pr-review skill (Step 5): When the docs-reviewer detects stale documentation, the skill now builds the connector, generates proposed metadata, and auto-updates the AUTO-GENERATED sections in docs/connector.mdx. Commits and pushes the changes to the PR branch.
  • pr-review.yaml: contents: write permission and expanded allowed tools to support doc auto-updates.

Why

Connector docs drift silently from code changes today. This enables the pr-review agent to automatically keep auto-generated doc sections (capabilities table, config params) in sync with the connector's actual metadata during PRs.

Test plan

  • Verify pr-review.yaml permissions and allowed tools are correct
  • Test pr-review skill Step 5 on a connector PR with changed capabilities
  • Verify scaffolded workflows have no syntax errors (act or manual review)

- publish-docs.yaml: scaffolded reusable workflow for publishing
  connector docs to registry on release (blocked by registry deployment)
- sync-stable-docs.yaml: scaffolded reusable workflow for syncing
  stable channel docs from registry to ConductorOne/docs
- release.yaml: add commented publish-docs job block
- pr-review skill: add Step 5 for auto-updating AUTO-GENERATED
  sections in docs/connector.mdx when docs-reviewer detects staleness
- pr-review.yaml: contents: write permission, expanded allowed tools
  for doc auto-update capability
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Feb 27, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch gontzess/automate-connector-docs

Comment @coderabbitai help to get the list of available commands and usage tips.

Assume PR won't merge until registry is live. Removed commented-out
patterns from publish-docs, sync-stable-docs, and release.yaml.
Kept TODO comments for registry endpoint configuration.
Doc freshness enforcement moves to CI check (check-doc-freshness job
in generate-baton-metadata) instead of agent-push. This avoids git
history pollution and keeps the pr-review agent read-only.
Moved to baton-admin where it runs as a cron job using the existing
baton-admin GitHub App for cross-repo commits to ConductorOne/docs.
@gontzess
Copy link
Copy Markdown
Contributor Author

Closing — doc publishing happens via the existing record-connector-registry step in release.yaml (RecordReleaseRequest already has a documentation field). No separate publish-docs workflow needed.

@gontzess gontzess closed this Feb 27, 2026
@gontzess gontzess deleted the gontzess/automate-connector-docs branch February 27, 2026 23:59
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.

1 participant