Skip to content

Prepare Ditto directory plugin disclosures and identifier - #27

Merged
Peyton-Spencer merged 2 commits into
mainfrom
task/plugin-directory-prep-20261107-002107
Nov 8, 2026
Merged

Peyton-Spencer merged 2 commits into
mainfrom
task/plugin-directory-prep-20261107-002107

Conversation

@Peyton-Spencer

Copy link
Copy Markdown
Contributor

Repo-only preparation for the Claude directory draft. Keeps displayName Ditto while proposing the manifest identifier heyditto, expands the README other-services disclosure, and clarifies that TDD test execution uses separately authorized host tools. Do not merge until Anthropic confirms whether changing the identifier preserves the existing draft. No connector slug or reviewer snapshot change; no submission or compliance acknowledgment.

@Peyton-Spencer
Peyton-Spencer marked this pull request as ready for review November 8, 2026 09:16
@Peyton-Spencer

Copy link
Copy Markdown
Contributor Author

@ditto review

@ditto-code

ditto-code Bot commented Nov 8, 2026 •

Copy link
Copy Markdown

✅ Ditto reviewed 298123a

See this review run → · no new comments · approved

Open in Ditto · 1m59s

@ditto-code ditto-code 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.

Ditto Review

No blocking issues found · risk: medium · 298123a

File coverage: 5 fully reviewed, 0 reviewed briefly, 0 not reviewed.

Rename the Claude plugin and clarify Ditto host capabilities — Renames the Claude-facing Ditto plugin identifier to heyditto and updates package validation to accept that name. The README and TDD skill clarify that external services and test execution depend on separately available, authorized host capabilities—not bundled plugin components. Review the identifier change for consistency with any consumers or publishing configuration outside these files.

Walkthrough (5 files)
File Change
plugins/ditto/.claude-plugin/plugin.json Changes the Claude plugin’s declared name from ditto to heyditto.
plugins/ditto/README.md Documents that integrations and tools beyond memory depend on host connections, permissions, and availability.
plugins/ditto/shared-skills.lock.json Updates the recorded hash for the changed TDD skill.
plugins/ditto/skills/tdd/SKILL.md Clarifies that running tests requires an authorized host execution tool and otherwise results remain unverified.
scripts/validate_package.py Adjusts manifest validation to expect heyditto for the Claude Ditto plugin while retaining other plugin names.
Review details
  • Models: review z-ai/glm-5.3-flash · verify openai/gpt-6-luna
  • Findings: 2 candidates → 0 verified → 0 posted
  • Withheld: rejected by verifier 2
  • Context index: 65 files · 2 changed symbols · 13 references
  • Tokens: 585434 in / 31669 out · $0.01
  • Run: 70f1acf0-2976-422a-a800-01cdf9c9aa91 · engine v2.0.0 · details
  • Ditto approves or requests changes in a separate review (blocking: 🟠 High and above). Inference is billed through your Ditto Router endpoint.

Open in Ditto · reply @ditto review to re-run · @ditto fix to push fixes

@ditto-code ditto-code 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.

Ditto: approved — no blocking findings (🟠 High or above) in a complete review of 298123a. Findings below that do not block Ditto’s approval.

@Peyton-Spencer
Peyton-Spencer merged commit 048f582 into main Nov 8, 2026
2 checks passed
@Peyton-Spencer
Peyton-Spencer deleted the task/plugin-directory-prep-20261107-002107 branch November 8, 2026 09:18
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