Skip to content

feat(research): publish original Skill size study - #601

Merged
harlan-zw merged 3 commits into
mainfrom
feat/skill-size-research
Oct 9, 2026
Merged

harlan-zw merged 3 commits into
mainfrom
feat/skill-size-research

Conversation

@harlan-zw

Copy link
Copy Markdown
Collaborator

🔗 Linked issue

❓ Type of change

  • 📖 Documentation
  • 🐞 Bug fix
  • 👌 Enhancement
  • ✨ New feature
  • 🧹 Chore
  • ⚠️ Breaking change

📚 Description

Make skilld's original research readable and reproducible, with an authoring guide as the next action. Adds a study of 12,141 Skills with accessible charts, repository and duplicate weighting, source downloads, dataset metadata, and a D1 snapshot of the findings. Also fixes the shared text component used to render branded social cards.

Research publication: frozen measurements produce the summary, accessible charts, downloads, and a persistent D1 snapshot

🤖 AI disclosure: Harlan Agent Kit modified this description. My AI open-source policy.

@harlan-github-agent harlan-github-agent Bot added harlan-agent-running An Agent holds a Task on this issue or pull request right now. harlan-agent-review-required Pull request triage requires an adversarial Review for this head commit. labels Oct 9, 2026
@harlan-github-agent

harlan-github-agent Bot commented Oct 9, 2026 •

Copy link
Copy Markdown
Contributor

🤖 MERGED

Harlan Agent Kit posted this automated review. It is not Harlan's personal review or approval. AI open source policy. Last updated: 2026-10-09 07:42 UTC.

GitHub merged this pull request.

  • Logged (40/100): The PR description image and both author verification screenshots (desktop chart, mobile layout) could not be visually inspected, so chart rendering, mobile table behavior, and overflow claims rest solely on the author's comment. View code Optional follow-up: Have a human reviewer inspect the three PNGs, or replace them with text/HTML evidence of the rendered charts before merge.
    • Ask an agent to verify and fix this finding
  • Logged (25/100): The claimed change to the shared OG text component behind every page's branded social card is completely unreviewed in this pass; if it alters shared rendering, all site OG images change silently. View code Optional follow-up: Name the changed component file in the PR description and re-run Review with diff evidence available (for example by excluding the binary research exports from diff computation).
    • Ask an agent to verify and fix this finding
  • Logged (20/100): No automated guard ties the committed measurements.json bytes to the recorded hash that the article's reproducibility claim rests on. View code Optional follow-up: Add the hash assertion as a unit test (and optionally a CI step) so a regenerated summary from edited inputs cannot ship self-consistently.
    • Ask an agent to verify and fix this finding
  • Logged (10/100): The chart component duplicates public snapshot data into the page chunk and requires a code edit for any future snapshot; a snapshot-agnostic module would decouple them. View code Optional follow-up: Move the imports behind a small shared module keyed by snapshot id, or fetch the JSON at runtime from its public URL.
    • Ask an agent to verify and fix this finding

Review completed. No finding above 80/100.

Selected findings run after merge and open separate pull requests.

@harlan-zw

harlan-zw commented Oct 9, 2026 •

Copy link
Copy Markdown
Collaborator Author

Agent verification: inspected desktop, mobile, and dark-mode charts in the built Cloudflare Worker. The research route renders charts and dataset metadata in server HTML, with a canonical URL and sitemap entry. Checked all 328,916 exported CSV cells against JSON, independently recalculated repository weighting, and queried the migrated local D1 snapshot. The branded social card returns a valid PNG.

Post-merge production verification for b202672: the site deploy job passed. The article serves the revised prose with the expected canonical and Dataset metadata. JSON, CSV, the authoring guide, and branded PNG return 200. The page has no JavaScript errors or horizontal overflow at 390px. The production D1 snapshot contains 12,141 Skills across 2,297 repositories, with the published measurement hash.

skill-research-built-chart

skill-research-built-mobile

@harlan-github-agent harlan-github-agent Bot added harlan-agent-running An Agent holds a Task on this issue or pull request right now. harlan-agent-review-required Pull request triage requires an adversarial Review for this head commit. and removed harlan-agent-running An Agent holds a Task on this issue or pull request right now. harlan-agent-review-required Pull request triage requires an adversarial Review for this head commit. labels Oct 9, 2026
@harlan-zw
harlan-zw merged commit b202672 into main Oct 9, 2026
8 checks passed
@harlan-github-agent harlan-github-agent Bot removed the harlan-agent-running An Agent holds a Task on this issue or pull request right now. label Oct 9, 2026
@harlan-zw
harlan-zw deleted the feat/skill-size-research branch October 9, 2026 07:39
@harlan-github-agent harlan-github-agent Bot removed the harlan-agent-review-required Pull request triage requires an adversarial Review for this head commit. label Oct 9, 2026
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.

1 participant