Repository navigation
Align proposal onboarding with the CompleteTech Skills package - #1
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Second-pass family alignment pushed at 604b97c. It synchronizes the eight shared files with the hub: branding/handoff and maintainer instructions, contributor/editor/PR standards, strengthened read-only validation and three-platform CI. Proposal templates, generators, dependencies, identities and approval gates are unchanged. Local common synthetic tests passed (10 parameterized methods); final CI and review are required before merge. No visual-render review is claimed. @codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 604b97c005
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| python scripts/validate_package.py | ||
| ``` | ||
|
|
||
| Also run the existing quality and regression commands in [README.md](README.md). Package checks are read-only and do not execute specialist generators. They validate metadata, required files, local documentation links and PNG chunk integrity, not decoded artwork or document truth. Do not weaken original Quality checks to obtain a green result. |
There was a problem hiding this comment.
Link the actual quality-check instructions
In a contributor checkout, README.md contains no quality or regression commands; the required python3 scripts/validate_quality.py command is documented in QUALITY.md. Following this new link therefore leaves contributors without the instructions for the mandated existing Quality gate, so point it to QUALITY.md or state the command directly.
AGENTS.md reference: AGENTS.md:L7-L7
Useful? React with 👍 / 👎.
Summary
Coordinated with CompleteTech-LLC/agentic-services-orchestrator-skill#1. Add family navigation, specialist onboarding, contributor guidance, skill-package.json, a read-only standard-library package validator, 17 synthetic regression tests, and independent least-privilege package CI.
Includes the hub review improvement: bounded PNG chunk/CRC validation and valid synthetic image fixtures. The first-run demo uses bundled Markdown and output/onboarding-demo.pdf rather than committed previews.
Compatibility
Preserve the existing README after navigation, templates, generators, SKILL.md, requirements, approval boundaries, brand assets, repository visibility, registry version, and original Quality workflow. No artifact generation is treated as approval or permission to publish.
Validation
The shared 17-test synthetic suite passed locally as part of the 24-test hub suite. Full-checkout package validation and existing Quality must pass on GitHub before merge. No local full-repository or visual-render verification is claimed; the execution environment cannot clone GitHub.