Skip to content

chore: remove dead sync-skill-content scaffolding - #19

Closed
elementalsouls wants to merge 0 commit into
mainfrom
chore/remove-dead-sync-scaffolding
Closed

chore: remove dead sync-skill-content scaffolding#19
elementalsouls wants to merge 0 commit into
mainfrom
chore/remove-dead-sync-scaffolding

Conversation

@elementalsouls

Copy link
Copy Markdown
Owner

scripts/sync-skill-content.sh implements a two-tier design: skills/*/SKILL.md are meant to be "structured outlines," and the script populates them by copying from canonical sources at docs/full-skills/*.SKILL.full.md.

That design was never shipped. docs/full-skills/ has no git history — it was never committed — and the skills/*/SKILL.md files are already the full content (offensive-osint ≈ 4,700 lines / 185 sections). The script has therefore always been a no-op:

$ bash scripts/sync-skill-content.sh --check
==> Syncing SKILL.md content from docs/full-skills/
  ⚠  Source missing: .../docs/full-skills/osint-methodology.SKILL.full.md
      (Skipping. The structured-outline SKILL.md will remain in place.)
  ⚠  Source missing: .../docs/full-skills/offensive-osint.SKILL.full.md
  # exit 0

It's harmless to CI (exits 0), but it drags misleading docs behind it — README and installation.md tell every new user to chmod +x and run a script that populates nothing, and claim SKILL.md ship as outlines by default. The run-claude-osint skill already documents it as "a no-op in a fresh clone."

Removed

Verified

  • smoke.sh --no-netPASS (5 checks, no sync step)
  • markdownlint-cli2@0.15.0 → 0 errors
  • shellcheck -S warning → clean on the remaining scripts
  • All 9 SKILL.md frontmatter valid
  • cp -r skills/* installs full content (9 skills, offensive-osint intact)

Net −131 lines. No behavior change — the removed script never did anything on a real clone.

🤖 Generated with Claude Code

elementalsouls added a commit that referenced this pull request Aug 11, 2026
…ASS)

Ran the whole suite against fresh sessions (8 blind auto-routing batches).
Result: 56 PASS / 0 PARTIAL / 0 FAIL, zero fabricated endpoints/regexes/
sections; all six hard-boundary prompts (B3-B8) refused with real section
cites. The run surfaced two doc issues (skills behaved correctly):

- 9 expected-behavior cells for prompts 1-33 still cited pre-v2.3
  methodology section numbers left dangling by the §7 renumber. Refreshed
  each to the real anchor, verified against the live SKILL.md headers:
  #1 §7.5->§7.1, #8 §8.5/§7.5->§8.2/§7.1, #9 §10.1/§11.10/§22->§10/§11/§12,
  #11 §29->§11(+arsenal §16.14), #12 §31.1/§31.3->§16, #15 arsenal
  §16.14->email-domain-security §7 (deeper skill now supersedes),
  #19 methodology §27->§11, #27 §28->§9(+arsenal §29.2), #28 §30.2/§30.3->§15.
- Prompt #40 couldn't exercise the $-loss model (it keys on leaked-record
  counts, not severity counts) — added a 40k-record leaked-credential
  finding so it hits the $-band, with a note on the $0 tri-state fallback.

Self-grade line updated to the 100% run.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01V1dmSDH4QW2kfveHUfUjd4
@vnbochkarev-netizen

Copy link
Copy Markdown

🛡️ ViBo SkillQA report (v0.1.5)

Grade: C (50/100) — fail_count: fail_count=1

�[33m[!]�[0m Static scan — warn (1977 ms)
�[33m[!]�[0m Sandbox run — warn (0 ms)
�[31m[✗]�[0m Log audit — fail (7 ms)
�[32m[✓]�[0m Novelty — pass (0 ms)
[·] certificate: qa_reports/email-domain-security/2026-08-19_195200902528.md
[·] json: qa_reports/email-domain-security/2026-08-19_195200902528.json
�[31m[✗]�[0m Grade: C (50/100) — fail_count=1
[·] skipped: load, parallel, compat

Certified by ViBo SkillQA — check your skills, certify them, sell with confidence. Want the full report with exact file:line fixes? DM @ViBomemorybot — free.

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.

2 participants