Repository navigation
fix: meet the portal's listing rules for the subtitle and the support URL - #75
Merged
Merged
Conversation
… URL The 1.2.2 draft reported 2 blocking findings: the subtitle must be 30 characters or fewer, and a support URL is required. The subtitle is now "Set up push, email, and SMS" (27 characters), and `supportURL` points at the repository issues page, the same URL the Claude manifest uses. The portal also advised against marketing claims and pricing text. The long description no longer says "market leading" or "free". The packager gate now checks the 2 submission limits (`displayName` and `shortDescription` at 30 characters) and the 4 listing URLs, with a self-test case for each new finding.
sherwinski
force-pushed
the
sherwin/directory-listing-rules
branch
from
October 15, 2026 16:59
7963f0f to
949d846
Compare
abdulraqeeb33
approved these changes
Oct 15, 2026
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
One Line Summary
.codex-plugin/plugin.jsonnow has a 27-character subtitle and asupportURL, and the packager gate checks both rules, so the OpenAI plugin directory draft can go to review.Motivation
The 1.2.2 draft uploaded (the name and MCP checks passed) and the portal reported 2 blocking findings on the metadata:
It also reported 2 advisory findings: "Remove or revise promises and guarantees that the plugin's functionality does not support" and "Remove pricing, subscription offers, and temporary promotions from the description." The long description said "market leading" and "A free OneSignal account".
Scope
.codex-plugin/plugin.json:shortDescription: "Set up push, email, and SMS" (27 characters). This keeps the intent of PR docs: name push, email, and SMS in the plugin descriptions #70, whose 36-character subtitle would fail the same check.supportURL:https://github.com/OneSignal/onesignal-agent-plugin/issues, the same URL the Claude manifest uses forsupportUrl.longDescription: "the market leading customer engagement solution" is now "a customer engagement platform"; "A free OneSignal account" is now "A OneSignal account".scripts/package_directory_bundle.py: the source gate checksdisplayNameandshortDescriptionat 30 characters (SUBMISSION_TEXT_LIMITS) and requires the 4 listing URLs as HTTPS (LISTING_URL_FIELDS). 2 new self-test cases cover a long subtitle and a missing support URL..mcp.json, or the Claude manifest.Testing
python3 -m json.tool .codex-plugin/plugin.jsonparsed with no errors.python3 -m py_compile scripts/*.pypassed.python3 scripts/package_directory_bundle.py --checkprintedbundle check passed (version 1.2.2; nothing written).python3 scripts/package_directory_bundle.py --self-testpassed all 19 cases, including the 2 new ones.Affected code
Checklist