Document that issues #455 and #456 were closed with maintainer guidance describing the required follow-up submission path before reopening. Refs #455 Refs #456
19 KiB
Maintenance Walkthrough - 2026-04-05
- Closed issues
#455and#456with maintainer comments explaining what a follow-up submission must include before reopening:- concrete repo diff or implementation PR
- source-only contributor branch
- Quality Bar checklist and maintainer validations from
.github/MAINTENANCE.md
- Reviewed open issues
#455and#456during the maintainer sweep; neither had a matching accepted PR and both remain open pending a source-quality contributor submission. - Triaged PR
#454as superseded by#457because#457rebuilds the Windows validation/test fixes on top of currentmainand includes the follow-up batch activation fix requested in review. - Verified PR
#457locally on the contributor head with:npm run validatenpm run validate:referencesnpm run check:warning-budgetnpm run check:readme-credits -- --base origin/main --head HEADnpm run testnpm run app:test:coveragenpm run app:build
- Cleaned PR
#457back to the repository's source-only PR contract by dropping maintainer-owned generated registry artifacts before merge review. - Normalized the PR metadata so the required Quality Bar Checklist is present before re-triggering the fork-based GitHub Actions checks.
Maintenance Walkthrough - 2026-03-30
- Merged PR #418 on GitHub with squash after approving the pending fork workflow run and waiting for
pr-policy,source-validation, andartifact-previewto finish green. - Repaired PR #423's stale metadata state by updating the PR body to include the required Quality Bar Checklist, then closed and reopened it to force fresh
pull_requestruns before squash merging it on GitHub. - Synced local
mainafter the PR merge batch so release preparation starts from the canonical remote state. - Resolved issue #421 by ensuring the
README.mdCommunity Contributed Skills section includesSoulPassonmain. - Resolved issue #419 by tightening the
github-issue-creatorfrontmatter description and "When to Use" guidance for better discoverability. - Prepared the
v9.3.0release notes inCHANGELOG.mdand recorded the maintainer actions here before running the release flow.
Maintenance Walkthrough - 2026-03-29
-
Re-triaged the full 2026-03-15 security finding set against current
mainand wrote a fresh current-head report indocs/maintainers/security-findings-triage-2026-03-29-refresh.md. -
Added a matching machine-readable export at
docs/maintainers/security-findings-triage-2026-03-29-refresh.csvso the refreshed statuses are available in both markdown and CSV form. -
Kept the old
2026-03-15markdown/CSV as historical baseline input, preserved the smaller2026-03-29addendum as a transition note, and pointed both docs at the new refresh as the current source of truth. -
The refreshed triage currently lands at:
0findings still present and exploitable0findings still present but low practical risk26obsolete/not reproducible on current HEAD7duplicates
-
The refresh folds in the hardening shipped today and earlier in the session:
- symlink/path safety in maintainer/install/web copy flows
- frontmatter parser robustness
- removal of shared frontend star writes
- secure Office unpack behavior
- migration away from predictable
/tmpstate files
-
Fixed the remaining production/documentation drift introduced by the web-app and CI hardening work:
- clarified that the hosted GitHub Pages app runs in static public-catalog mode
- documented that
Sync Skillsis development-only unless explicitly enabled in local maintainer runs - documented that web-app save/star interactions are intentionally browser-local today
-
Hardened the maintainer documentation so release and CI expectations now match the live workflows:
- release docs now mention the shared
tools/requirements.txtinstall path, the web-app coverage gate, and blockingnpm audit --audit-level=highon publish - maintainer docs now document the narrow canonical-artifact auto-sync contract on
main
- release docs now mention the shared
-
Expanded the documented risk-maintenance workflow after the new automation landed:
audit:skillsexposessuggested_risksync:risk-labelssupports conservative high-confidence legacy cleanup- offensive auto-promotions now also insert the canonical
AUTHORIZED USE ONLYnotice
-
Updated user-facing install docs to mention that the npm installer now uses a shallow clone for lighter first-run installs.
-
Updated the onboarding/trust docs to reflect the real
risktaxonomy (unknown,none,safe,critical,offensive) instead of the older simplified wording.
Maintenance Walkthrough - 2026-03-25
- Imported 14 skills from Dimillian/Skills into
skills/:app-store-changeloggithubios-debugger-agentmacos-menubar-tuist-appmacos-spm-app-packagingorchestrate-batch-refactorproject-skill-auditreact-component-performancesimplify-codeswift-concurrency-expertswiftui-liquid-glassswiftui-performance-auditswiftui-ui-patternsswiftui-view-refactor
- Normalized the imported skill metadata to match repository validation requirements:
- shortened oversized frontmatter descriptions
- added
risk,source, anddate_added - added
## When to Usesections so the imported batch does not increase the warning budget
- Added source attribution for
Dimillian/Skillsin:README.mdunderCredits & Sourcesdocs/sources/sources.md
- Merged PR
#395via GitHub squash merge after maintainer refresh of forked workflow approvals and PR body normalization; this added the newsnowflake-developmentskill. - Merged PR
#394via GitHub squash merge after converting the contributor branch back to source-only, normalizing the PR checklist body, and shortening an oversizedwordpress-penetration-testingdescription so CI passed. - Patched
skills/snowflake-development/SKILL.mdonmainwith a## When to Usesection so the repository stayed within the frozen validation warning budget after the PR merge batch. - Reworked
/apply-optimizeautomation to address GitHub code scanning alert#36: the publicissue_commenttrigger now only queues a trusted workflow, while the privileged branch checkout/apply logic runs in a separateworkflow_dispatchpath limited to same-repository branches. - Ran the required direct-
mainmaintainer sync flow after touchingskills/:npm run chainnpm run check:warning-budgetnpm run catalog
- Synced maintainer-owned generated artifacts and metadata to the new
1,325+skill count:README.mdpackage.jsonskills_index.jsonCATALOG.mddata/catalog.jsondata/bundles.json- curated user/maintainer docs updated by
sync_repo_metadata.py
Maintenance Walkthrough - 2026-03-21
- Imported and normalized a new batch of external skills into
skills/, covering Anthropic Claude API/internal comms entries, marketing workflows, SEO orchestration/sub-skills, and Obsidian-focused file-format/CLI skills. - Added and standardized the following imported skill families:
claude-api,internal-commsad-creative,ai-seo,churn-prevention,cold-email,content-strategy,lead-magnets,product-marketing-context,revops,sales-enablement,site-architectureseo,seo-competitor-pages,seo-content,seo-dataforseo,seo-geo,seo-hreflang,seo-image-gen,seo-images,seo-page,seo-plan,seo-programmatic,seo-schema,seo-sitemap,seo-technicaldefuddle,json-canvas,obsidian-bases,obsidian-cli,obsidian-markdown
- Preserved the existing
docx,pdf,pptx, andxlsxaliases as the repository's symlinked*-officialentries instead of duplicating those directories. - Normalized imported frontmatter so the new skills align with repository validation expectations:
- shortened oversized descriptions
- added missing
risk,source, anddate_addedfields where needed - added
## When to Usesections across the new imports - removed or rewrote imported dangling links that referenced non-existent upstream paths in this repository
- Added maintainer provenance notes in
docs/maintainers/skills-import-2026-03-21.mdso the source repository for each imported skill group is documented for future maintenance. - Regenerated maintainer-owned derived artifacts after the import:
README.mdskills_index.jsonCATALOG.mddata/catalog.jsondata/bundles.json
- Verified the direct-
mainmaintenance flow with:npm run validatenpm run indexnpm run catalognpm run chain
Maintenance Walkthrough - 2026-03-18
- Fixed issue
#344by correcting.claude-plugin/marketplace.jsonso the marketplace plugin entry usessource: "./"instead of".", matching Claude Code's relative-path schema requirement for marketplace entries. - Added
tools/scripts/tests/claude_plugin_marketplace.test.jsand wired it into the local test suite so invalid marketplacesourcepaths fail fast in CI/maintainer verification. - Merged PRs
#333,#336,#338,#343,#340,#334, and#345via GitHub squash merge after maintainer refresh of forked workflows and PR metadata. - Closed PR
#337and PR#342as superseded by#338, then closed issue#339manually after confirming the accepted fix path; issue#335auto-closed from the merged PR body. - Closed issue
#344with a follow-up comment after shipping the plugin marketplace fix onmain, and left PR#341open with a blocking review comment because the submitted skill content is corrupted even though CI is green. - Documented a new maintainer edge case in
.github/MAINTENANCE.md: forked runs inaction_required,pr-policyfailures caused by stale PR bodies, the REST API fallback whengh pr editfails with the Projects Classic GraphQL error, and the need toclose/reopena PR when a plain rerun does not pick up updated metadata. - Refreshed the release-facing docs for
8.2.0acrossREADME.md,docs/users/getting-started.md,docs/users/walkthrough.md, andCHANGELOG.md. - Published release
v8.2.0onmainwith:npm run release:preflightnpm run security:docsnpm run release:prepare -- 8.2.0npm run release:publish -- 8.2.0
Maintenance Walkthrough - 2026-03-17
- Synced
mainafter the six merged community PRs and re-verified all forked PR workflows through GitHub before final release prep. - Reopened/approved forked GitHub Actions runs where needed, normalized missing PR quality checklists, and merged PRs
#331,#330,#326,#324,#325, and#329with GitHub squash merge. - Patched
skills/vibers-code-review/SKILL.mdon the contributor branch for PR#325so the skill had valid YAML frontmatter, aWhen to Usesection, and explicit limitations; reran CI and merged after green checks. - Closed issue
#327with a release comment pointing to#331, and closed issue#328as a duplicate of#269with links to the README recovery guidance anddocs/users/windows-truncation-recovery.md. - Updated release-facing docs before cutting
v8.1.0:README.mddocs/users/getting-started.mdCHANGELOG.mdwalkthrough.md
- Refreshed the README contributor acknowledgements to include the latest merged contributors from the maintenance batch.
- Release workflow to run for
8.1.0:npm run release:preflightnpm run security:docsnpm run release:prepare -- 8.1.0npm run release:publish -- 8.1.0
Maintenance Walkthrough - 2026-03-12
- Merged PRs
#277,#272,#275,#278, and#271via GitHub squash merge after bringing contributor branches into a mergeable state and refreshing PR bodies against the quality checklist in.github/MAINTENANCE.md. - Verified PR
#271locally withnpm run validate:referencesandnpm run testbefore merge; confirmed#269auto-closed from the merged PR body. - Added a user-facing Windows truncation recovery guide at
docs/users/windows-truncation-recovery.md, linked it fromREADME.md,docs/users/faq.md,docs/users/getting-started.md, anddocs/integrations/jetski-cortex.md, and credited the workflow to issue#274. - Updated
skills/metasploit-framework/SKILL.mdto remove the remote installer flow, require an existing Metasploit installation, and add the required offensive-skill warning. - Refreshed
README.mdto remove stale7.2.0/7.4.0onboarding copy, align the star badge with the current milestone, and fix the TOC link for## Contributing. - Normalized the active English docs (
README.md, user guides, Kiro guide, and evergreen maintainer docs) to the current7.6.0/1,250+ skillsstate and removed emoji from H2 headers where maintenance rules require clean anchors. - Ran the required maintenance validations after the direct fixes:
npm run validatenpm run validate:referencesnpm run chainnpm run catalog
- Final release prep, issue closure comments, and verification were completed on
main.
Maintenance Walkthrough - 2026-03-13
- Fixed
tools/scripts/update_readme.pyso normalnpm run readmeruns preserve the existingregistry-syncstar/timestamp values instead of rewriting them on every execution, which was causing non-deterministic PR drift failures in CI. - Updated
tools/scripts/sync_repo_metadata.pyto expose the same explicit--refresh-volatilebehavior for live star/timestamp refreshes, keeping release/metadata refresh flows available without destabilizing contributor PR checks. - Updated
.github/workflows/ci.ymlso generated registry drift is informational on pull requests but still strict onmain, with auto-sync remaining the canonical path for shared artifacts after merge. - Updated
.github/MAINTENANCE.md,docs/maintainers/ci-drift-fix.md, anddocs/maintainers/merging-prs.mdto document the lower-friction merge flow: validate source changes on PRs, keepmainfor generated conflicts, and letmainauto-sync the final artifact set. - Verified the fix with:
python3 tools/scripts/update_readme.py --dry-runpython3 tools/scripts/sync_repo_metadata.py --dry-runnpm run readmenpm run validate:references
- Added
tools/config/generated-files.jsonas the single contract for derived registry artifacts so CI, maintainer scripts, and docs share the same file list. - Added scripted workflow entrypoints:
npm run pr:preflight,npm run release:preflight,npm run release:prepare -- X.Y.Z, andnpm run release:publish -- X.Y.Z. - Split PR CI into
pr-policy,source-validation, andartifact-previewso PRs stay source-only, policy failures are explicit, and generated drift is previewed separately from source validation. - Updated
CONTRIBUTING.mdand.github/PULL_REQUEST_TEMPLATE.mdso contributors are told not to commit derived files and to enableAllow edits from maintainers.
Maintenance Walkthrough - 2026-03-14
- Added root Claude Code plugin marketplace support via
.claude-plugin/plugin.jsonand.claude-plugin/marketplace.json, exposing the repository as a single plugin entry that points at the existingskills/tree. - Updated the user onboarding trinity (
README.md,docs/users/getting-started.md,docs/users/faq.md) so Claude Code users can install via/plugin marketplace add sickn33/antigravity-awesome-skillsin addition to the existingnpxinstaller flow. - Merged PRs
#302,#301,#299,#297,#296,#287,#298, and#293via GitHub squash merge after maintainer preflight, including a maintained follow-up commit on the contributor branch for#298and a maintainer conflict-resolution refresh on#293. - Verified the issue-driven fixes locally before merge:
#301:python3 -m py_compile skills/notebooklm/scripts/browser_utils.py#299:node -c tools/bin/install.js
- Verified the skill/docs PRs locally before merge:
#297,#296,#287,#298:npm run validate#293,#298:npm run validate:references
- Closed issues
#288,#300,#286, and#281from the merged fixes and release notes flow; documented#294as a release follow-up because the support already exists in the current catalog. - Removed stale Windows
core.symlinks=true/ Developer Mode guidance from the user docs after the#299installer fix, keeping the Windows path on the standard clone/install flow. - Ran the post-merge maintainer sync on
main:npm run chainnpm run catalog
- Refreshed
CHANGELOG.md,README.md,docs/users/getting-started.md,docs/users/faq.md, and the contributor acknowledgements to prepare the single7.8.0release cut.
Maintenance Walkthrough - 2026-03-21
- Imported the missing external skill coverage identified from
travisvn/awesome-claude-skills,anthropics/skills,coreyhaines31/marketingskills,AgriciDaniel/claude-seo, andkepano/obsidian-skills, bringing the indexed registry to1,304skills onmain. - Added maintainer attribution notes in
docs/maintainers/skills-import-2026-03-21.mdand refreshed the generated registry artifacts after the import batch. - Re-aligned the public documentation surface to the current repository state:
README.mdpackage.jsondocs/users/getting-started.mddocs/users/usage.mddocs/users/claude-code-skills.mddocs/users/gemini-cli-skills.mddocs/users/visual-guide.mddocs/users/bundles.mddocs/users/kiro-integration.mddocs/integrations/jetski-cortex.mddocs/maintainers/repo-growth-seo.mddocs/maintainers/skills-update-guide.md
- Updated the changelog
Unreleasedsection so the post-v8.4.0main branch state documents both the imported skill families and the docs/About realignment. - Automated the recurring docs metadata maintenance by extending
tools/scripts/sync_repo_metadata.py, wiring it intonpm run chain, and adding a regression test so future skill-count/version updates propagate through the curated docs surface without manual patching. - Added a remote GitHub About sync path (
npm run sync:github-about) backed bygh repo edit+gh api .../topicsso the public repository metadata can be refreshed from the same source of truth on demand. - Added maintainer automation for repo-state hygiene:
sync:contributorsupdates the README contributor list from GitHub contributors,check:stale-claims/audit:consistencycatch drift in count-sensitive docs, andsync:repo-statenow chains the local maintainer sweep into a single command. - Hardened automation surfaces beyond the local CLI:
mainCI now runs the unified repo-state sync, tracked web artifacts are refreshed throughsync:web-assets, release verification now uses a deterministicsync:release-statepath plusnpm pack --dry-run, the npm publish workflow reruns those checks before publishing, and a weeklyRepo HygieneGitHub Actions workflow now sweeps slow drift onmain. - Added two maintainer niceties on top of the hardening work:
check:warning-budgetfreezes the accepted135validation warnings so they cannot silently grow, andaudit:maintainerprints a read-only health snapshot of warning budget, consistency drift, and git cleanliness.