Skip to content

feat: add onboard skill for project onboarding & guided learning - #5448

Open
Keviniscool-boy wants to merge 1 commit into
github:mainfrom
Keviniscool-boy:add-onboard-skill
Open

Keviniscool-boy wants to merge 1 commit into
github:mainfrom
Keviniscool-boy:add-onboard-skill

Conversation

@Keviniscool-boy

Copy link
Copy Markdown

Summary

Adds onboard, a self-contained skill that helps developers onboard into a new codebase, company, or any pile of mixed handover materials (code repo, docs, PDF/Excel/PPT):

  • Identifies the material type, then decomposes how everything works — every claim carries a source citation or is flagged as inference; gaps and contradictions are surfaced explicitly, never silently filled
  • Builds a goal-driven three-tier learning path (must-know now / lookup-when-needed / skippable) around the user's actual goal
  • Teaches interactively by default; gaps exposed during teaching are back-filled into the decomposition, so teaching doubles as a second verification pass
  • Read-only by default: never writes source files, no side-effect commands, never reads .env/keys

What's included

  • skills/onboard/SKILL.md — the full skill
  • skills/onboard/scripts/inventory.mjs — optional zero-dependency read-only inventory helper
  • npm run skill:validate and npm run build both pass; generated docs/README.*.md tables and .github/plugin/marketplace.json are updated in this PR

Validation evidence

Every rule in the SKILL.md was validated by running stranger agents (fresh session, zero context) on real materials: a large code repo, an unstructured 244KB transcript, real banking regulations (Word/PDF/Excel), a PPT deck, and a 703-file repo with sampling. Full test ledger in evals/. MIT licensed.

@Keviniscool-boy
Keviniscool-boy requested review from a team as code owners October 24, 2026 16:28
Copilot AI max review requested due to automatic review settings October 24, 2026 16:28

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

@github-actions github-actions Bot added new-submission PR adds at least one new contribution plugin PR touches plugins skills PR touches skills labels Oct 24, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🔒 PR Risk Scan Results

Scanned 3 changed file(s).

Severity Count
🔴 High 0
🟠 Medium 0
ℹ️ Info 1
Severity Rule File Line Match
ℹ️ skill-script-touched skills/onboard/scripts/inventory.mjs 1 skills/onboard/scripts/inventory.mjs

This is an automated soft-gate report. Findings indicate review targets and do not block merge by themselves.

@github-actions

Copy link
Copy Markdown
Contributor

🟡 Contributor Reputation Check: MEDIUM risk

Check Risk
Profile MEDIUM
Credential audit NONE

Maintainers: please review this contributor before merging.
See the workflow run for full details.
Automated check powered by AGT.

@github-actions github-actions Bot added needs-review:MEDIUM Contributor reputation check flagged MEDIUM risk awaiting-automation merge-risk:high labels Oct 24, 2026
@github-actions

github-actions Bot commented Oct 24, 2026 •

Copy link
Copy Markdown
Contributor

🚦 Submission status: 🛠️ Requires submitter fixes

Risk tier: merge-risk:high — Privileged execution, automation, or review-policy change
Required to merge: passing submission-gate checks plus 1 approval from reviewers with write access, including a maintainer with admin or maintain permission.

Why this tier
  • skills/onboard/scripts/inventory.mjs is a high-risk path (automation, scripts, MCP config, hooks, or review policy)
  • Spawns processes or evaluates code in skills/onboard/scripts/inventory.mjs

Automated checks

Check Status Details
Line endings ✅ Passed Passed · logs
Spelling ✅ Passed Passed · logs
Generated README consistency ❌ Contribution failure Failed step: Fail workflow if files need updating · logs
Skill validation ✅ Passed Passed · logs
Skill lint (vally) ✅ Passed Passed · logs
Risk scan ✅ Passed Passed · logs
Contributor reputation ✅ Passed Passed · logs
Duplicate resource scan ✅ Passed Passed · logs
PR quality signal ⏭️ Skipped Skipped by its workflow · logs

Action needed

  • Generated README consistency failed. Run npm start locally and commit the regenerated files.

Review

  • Approvals: 0/1
  • Assigned reviewer: not assigned yet — comment /request-review to ask for one
  • Review target date: not set
  • Still needed: 1 more approval(s); an approval from a maintainer with admin or maintain permission
  • The core-maintainers pool is not staffed yet; an approver with admin or maintain permission is required instead.

Commands

Command Who What it does
/rerun-checks PR author, maintainers Re-runs failed or incomplete checks and re-evaluates this gate
/request-review PR author, maintainers Asks the review rotation to assign a reviewer (adds needs-reviewer)

Updated for ecdb7c3 · gate run · This comment is maintained automatically — see submission gate docs.

@github-actions github-actions Bot added requires-submitter-fixes Submission has quality-gate findings that submitter must fix before maintainer review and removed awaiting-automation labels Oct 24, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🔍 Vally Lint Results

✅ All checks passed

Scope Checked
Skills 1
Agents 0
Total 1
Severity Count
❌ Errors 0
⚠️ Warnings 0
ℹ️ Advisories 0

Summary

Level Finding
ℹ️ ✅ onboard (2/2 checks passed)
ℹ️ ✓ [spec-compliance] All 1 skill(s) are spec-compliant.
ℹ️ ✓ spec-compliance: All spec checks passed.
ℹ️ ✓ [valid-refs] All file references across 1 skill(s) are valid.
ℹ️ ✓ valid-refs: All file references resolve to existing files within the skill directory.
ℹ️ 1 skill(s) linted, 1 passed
Full linter output
### Linting skills/onboard
✅ onboard (2/2 checks passed)
    ✓ [spec-compliance] All 1 skill(s) are spec-compliant.
        ✓ spec-compliance: All spec checks passed.
    ✓ [valid-refs] All file references across 1 skill(s) are valid.
        ✓ valid-refs: All file references resolve to existing files within the skill directory.

1 skill(s) linted, 1 passed

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merge-risk:high needs-review:MEDIUM Contributor reputation check flagged MEDIUM risk new-submission PR adds at least one new contribution plugin PR touches plugins requires-submitter-fixes Submission has quality-gate findings that submitter must fix before maintainer review skills PR touches skills

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants