Skip to content

ADFA-6318: Fix repo and plugin names in plugin-api.md - #2103

Merged
hal-eisen-adfa merged 3 commits into
stagefrom
docs/ADFA-6318
Oct 6, 2026
Merged

hal-eisen-adfa merged 3 commits into
stagefrom
docs/ADFA-6318

Conversation

@hal-eisen-adfa

@hal-eisen-adfa hal-eisen-adfa commented Oct 5, 2026 •

Copy link
Copy Markdown
Collaborator

ADFA-6318

Brings docs/plugin-api.md in line with the plugin names in appdevforall/addons (formerly plugin-examples). Under the current naming standard, plugin names no longer end in -plugin.

  • plugin-examples -> addons (gh api repos/appdevforall/plugin-examples resolves to appdevforall/addons).
  • apk-viewer-plugin/, markdown-preview-plugin/, keystore-generator-plugin/ -> APK-Analyzer, Markdown-Previewer, Keystore-Generator.
  • ai-core / AI Core -> AI-Core.

Checked: no other plugin names appear in the file. REVIEW.md:218 and a few other docs still use the old names. I left those alone; this PR changes plugin-api.md only.

The second half of the ticket (the AI-Agent-OpenAI README) lives in the addons repo and gets its own PR.

appdevforall/plugin-examples was renamed to appdevforall/addons.

@claude claude Bot 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.

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review for a one-time review, or @claude review always to subscribe this PR to a review on every future push.

Tip: disable this comment in your organization's Code Review settings.

@coderabbitai

coderabbitai Bot commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Essentials
  • Run ID: 7e41c5ce-7da6-4415-93f5-a46e667c3b0a
📥 Commits

Reviewing files that changed from the base of the PR and between 03384d4 and f59afb2.

📒 Files selected for processing (1)
  • docs/plugin-api.md

Included review availability: This review used your included allowance. 3 included reviews remain after this review. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour.


📝 Summary
  • Updated the impact-check checklist in docs/plugin-api.md to refer to appdevforall/addons instead of plugin-examples.
  • No risks or best-practice violations are evident from this documentation-only change.

Walkthrough

The plugin API documentation now uses “AI-Core” consistently in service descriptions and clarifies that plugin-implemented service interfaces include ToolSource. The impact-check checklist now refers to in-tree example plugins and the external addons repository.

Changes

Plugin API documentation

Layer / File(s) Summary
Service guidance and impact-check checklist
docs/plugin-api.md
Cross-plugin service descriptions and interface guidance use “AI-Core.” The checklist refers to plugins in the external addons repository and names AI-Core and Keystore-Generator as examples.

Priority: ⬇️ Low

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

Change: Other

Merge Risk: ⚪ Minimal · up to f59af

No actionable merge-blocking risk is identified in this documentation update.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title clearly identifies the change to repository and plugin names in plugin-api.md.
Description check ✅ Passed The description explains the naming updates in docs/plugin-api.md and identifies related work that is out of scope.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

A rabbit reads the plugin guide,
“AI-Core” now stands beside.
ToolSource joins the service line,
Addons examples read just fine.
I nibble clover, hop, and hide.

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

Name AI-Core as its addons plugins/ directory does (was ai-core / AI Core).
Drop keystore-generator-plugin from the in-tree list: ADFA-4659 removed it,
and it now lives in addons as plugins/Keystore-Generator.
@hal-eisen-adfa hal-eisen-adfa changed the title ADFA-6318: Point plugin-api.md at the renamed addons repo ADFA-6318: Fix repo and plugin names in plugin-api.md Oct 5, 2026
The plugin naming standard dropped the -plugin suffix: apk-viewer-plugin,
markdown-preview-plugin and keystore-generator-plugin are now APK-Analyzer,
Markdown-Previewer and Keystore-Generator under addons plugins/.
@hal-eisen-adfa
hal-eisen-adfa merged commit 1c33961 into stage Oct 6, 2026
9 checks passed
@hal-eisen-adfa
hal-eisen-adfa deleted the docs/ADFA-6318 branch October 6, 2026 00:55
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