Repository navigation
Conversation
A badge that links to a noindex hub earns nothing, so the hub hides the badge block unless seoIndexable. The brand kit page is a tool page, so it is now noindex.
Collaborator
Author
|
Checked in the dev server: /b/antfu/skills/vite returns 200, image/svg+xml, 1 day cache. Indexable hub /gh/antfu/skills shows the block with the canonical link. Noindex hubs (torvalds/linux, vercel/next.js) show none. Hub block:
|
harlan-github-agent
Bot
added
harlan-agent-running
Contributor
🤖 BLOCKED
Selected findings run after merge and open separate pull requests. |
Collaborator
Author
🤖 READY · 95/100
|
harlan-zw
added
harlan-agent-ready
harlan-github-agent
Bot
added
harlan-agent-running
harlan-zw
marked this pull request as draft
September 30, 2026 14:32
harlan-zw
added a commit
that referenced
this pull request
Sep 30, 2026
harlan-github-agent
Bot
added
harlan-agent-running
![harlan-github-agent[bot]](https://avatars.githubusercontent.com/in/4579275?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Harlan Agent Kit posted this automated review. AI open source policy.
Automated Review: BLOCKED.
harlan-github-agent
Bot
added
harlan-agent-pending
harlan-github-agent
Bot
added
harlan-agent-ready
1 of 6 tasks
harlan-zw
added a commit
that referenced
this pull request
Oct 7, 2026
* test(helpers): load each migrated sqlite database from an image Most suites build a fresh database per test and replayed every migration each time, about 100 ms per database. Replay each migration list once per file and deserialize the image after that. Node before 24.16 has no serialize, so it keeps the replay. * test: delete tests that restate source, copy, or config These files read source or docs and grep the text, assert copy and markup, restate constant tables, or repeat another file's coverage. None of them can catch a behaviour bug that typecheck, lint, or a remaining test would miss. * test: boot Nuxt only for tests that need it Every file booted a Nuxt app, though about 40 of 377 use it. The default environment is now node. Files that mount components or read Nuxt runtime config carry the .nuxt.test.ts suffix, so defineVitestConfig runs them in its nuxt project. The node project runs in vmThreads and drops the @nuxt/test-utils setup file, which loaded Vue and its compiler in every file. Workers now follow the CPU quota: 2 workers under the 4-CPU CI quota left half of it idle. Also removes single cases that grep source, assert copy or class names, or restate constant tables, and the node environment pragmas that now repeat the default. * test: keep pragmas in files that open branches also edit Open branches edit these four files. Leaving their redundant node pragma in place keeps those merges free of conflicts. * test: keep the brand kit test that an open draft extends Draft #321 adds a noindex case to this file. Deleting it would leave that draft a modify and delete conflict. * test: shrink the streamed pack fixture At 300 KiB the 7-byte chunk case timed out under the full suite's worker count. 20 KiB still crosses many tar blocks at every chunk size.
harlan-github-agent
Bot
added
harlan-agent-blocked
This was referenced Oct 8, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
❓ Type of change
📚 Description
For experiment B (links and demand), Skill authors need a README badge that links to their skilld page. The badge SVG (
/b/<owner>/<repo>[/<skill>]) and the copyable snippet already existed. A badge that points at a noindex hub earns nothing, so the repository hub now shows the badge block only when the hub is indexable./brand-kit/github-badgewasindex,follow; it is a tool page and is nownoindex,follow. Its sitemap listing is left to the sitemap work.I added no referral tracking. GitHub referrals already show as a referrer in Cloudflare analytics, and a
?ref=on the link would split canonical URLs.