Skip to content

Docs: Fix broken links described in issue 679 - #742

Merged
esnible merged 1 commit into
rossoctl:mainfrom
esnible:fix-broken-link
Aug 10, 2026
Merged

Docs: Fix broken links described in issue 679#742
esnible merged 1 commit into
rossoctl:mainfrom
esnible:fix-broken-link

Conversation

@esnible

@esnible esnible commented Aug 10, 2026

Copy link
Copy Markdown
Member

Summary

Resolves #679 by changing links to match current state of CPEX repo.

Summary by CodeRabbit

  • Documentation
    • Updated CPEX documentation links to point to the standalone CPEX repository.
    • Replaced outdated links to the former plugins framework repository.

Signed-off-by: Ed Snible <snible@us.ibm.com>
@coderabbitai

coderabbitai Bot commented Aug 10, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 6e311395-9523-4a8f-97fa-3e81f9e90224

📥 Commits

Reviewing files that changed from the base of the PR and between 29cb0f5 and bd07b08.

📒 Files selected for processing (1)
  • docs/proposals/authbridge-hooks.md

📝 Walkthrough

Walkthrough

Updated three CPEX documentation links in docs/proposals/authbridge-hooks.md to use the standalone contextforge-org/cpex repository.

Changes

CPEX documentation links

Layer / File(s) Summary
Update CPEX references
docs/proposals/authbridge-hooks.md
The embedded project, specification, Go API, and terminology links now reference the standalone CPEX repository.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Suggested reviewers: huang195, osherelhadad

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the documentation link fix and references issue #679.
Linked Issues check ✅ Passed The changes replace the broken CPEX repository links with links to the standalone repository, addressing issue #679.
Out of Scope Changes check ✅ Passed The pull request changes only the CPEX documentation links identified in issue #679.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

@esnible esnible added the ready-for-ai-review Request automated AI code review from clawgenti label Aug 10, 2026

@clawgenti clawgenti left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Docs-only link fix updating three CPEX references from the old contextforge-plugins-framework repo to the new contextforge-org/cpex repo. Verified that both target files (plugin-framework-spec.md and cpex-go-spec.md) exist at the new paths.

All checks pass. Ready for human review.


Reviewed by clawgenti using the github-pr-review skill

@esnible esnible added the ready-for-human-review AI review passed, ready for human reviewer label Aug 10, 2026
@esnible
esnible merged commit 91b5677 into rossoctl:main Aug 10, 2026
21 checks passed
@github-project-automation github-project-automation Bot moved this from New/ToDo to Done in Rossoctl Issue Prioritization Aug 10, 2026
@esnible
esnible deleted the fix-broken-link branch August 10, 2026 18:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-for-ai-review Request automated AI code review from clawgenti ready-for-human-review AI review passed, ready for human reviewer

Projects

Status: Done

4 participants