Skip to content

ci: fix documentation notifier destination repository - #43

Closed
uniskela wants to merge 2 commits into
mainfrom
ci/fix-docs-target-com-20261009
Closed

uniskela wants to merge 2 commits into
mainfrom
ci/fix-docs-target-com-20261009

Conversation

@uniskela

@uniskela uniskela commented Oct 9, 2026

Copy link
Copy Markdown
Owner

Root cause

The docs notifier selected non-existent uniskela/.com for GitHub App installation and repository_dispatch. The Stack Manager release workflow already calls this notifier as a reusable workflow.

Fix

  • Use repositories: com for the short-lived GitHub App token instead of .com.
  • Dispatch docs-source-updated to repos/uniskela/com/dispatches.
  • Update the associated maintainer documentation to name the real destination.

Test plan

  • Check CodeRabbit and required CI checks.
  • Merge the receiver fix in uniskela/com first.
  • Merge this PR and run Notify Uniskela documentation manually on main.
  • Confirm successful token creation and delivery to Propose documentation updates in uniskela/com.

CI/docs-only change, no product release; leave unmerged until reviewed.

@coderabbitai

coderabbitai Bot commented Oct 9, 2026

Copy link
Copy Markdown
Contributor

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 5d7ea1dd-78c4-4239-bd90-f520be9375bb
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

uniskela commented Oct 9, 2026

Copy link
Copy Markdown
Owner Author

Closing as superseded by #44, which makes the same uniskela/com destination fix on the current main (after the docs reorganisation in #42). This branch was cut before #42 and would revert the docs/public paths in .github/docs-sync.md.

The UI test failure here was an intermittent axe document-title race in the E2E helper, not caused by this change. It is fixed separately in #45. The remaining header-comment reference to uniskela/.com is now fixed on #44.

🤖 Generated with Claude Code


Generated by Claude Code

@uniskela uniskela closed this Oct 9, 2026
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