Skip to content

feat(trending): put braille sparks and run chips on the board - #431

Merged
harlan-zw merged 2 commits into
mainfrom
feat/brand-trending
Oct 5, 2026
Merged

harlan-zw merged 2 commits into
mainfrom
feat/brand-trending

Conversation

@harlan-zw

@harlan-zw harlan-zw commented Oct 5, 2026 •

Copy link
Copy Markdown
Collaborator

🔗 Linked issue

Related to #427 and #426

❓ Type of change

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

📚 Description

The dot brand from #427 needs to reach /skills/trending, the board readers copy from. Each row now draws the signal that ranked it and prints its command in the brand shapes.

Architecture: the trending loader counts mentions per day, the feed carries them and repoSkillCount to the board, and the board row draws the spark and the run chip

  • A row with posts shows a braille spark of counted mentions over seven rolling 24-hour days. loadTrendingSkills already reads those posts, so /api/feed/trending gains mentionsByDay from the same rows that rank the Skill. A row with no mention in seven days shows no spark; its posts carry their own dates.
  • Surge and filler rows keep the star line, because stars ranked them. The surge reason now reads Star surge: +N stars in a day.
  • A compact run chip appears only when the row stands for exactly one Skill. Filler and the all-time range pick one Skill out of a repository, so a repository with several Skills gets no chip. The feed's fallback items gain repoSkillCount for that check. Copies report through useInstallCopy with surface trending-row.
  • The header aside carries Teach your agent skilld with the skilld install chip at every width, under the weekly invitation on wide screens.
  • The empty state shows the converge texture.
  • The nav and the /skills table replace the flame emoji with a braille mark. The flame animation CSS is gone. The nav mark is stone, because the logo dot is the header's rose.
  • Ranks print in stone dots, so each row's rose belongs to its motifs.

Page question: what are devs talking about, and how do I run it? Primary action: copy the run command. Owning concern: Loop 1. VISION principle 4: PASS; each number says what it counts, and a star-ranked row never shows a mentions spark. DESIGN motifs: PASS; one rose per motif, and the nav mark draws no rose.

Gap: the board has no per-Skill change dates, only a repository push date on the all-time range. That range keeps Updated {date} instead of a change grid.

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

The new brand puts the dot motifs where the registry's evidence lives.
Trending is the board readers copy from, so each row now shows its signal
and its command in the brand shapes.

- Each row with posts draws seven days of counted mentions as a braille
  spark. The loader already reads every post in the window, so the counts
  come from the same rows that rank the Skill, in rolling 24-hour days.
- Surge and filler rows keep the star line, because stars ranked them.
  The surge reason now says it was a star surge, as VISION principle 4
  asks of every trending number.
- A compact run chip appears only where the row stands for exactly one
  Skill. Filler and the all-time range pick a Skill out of a repository,
  so a multi-Skill repository gets no command. Copies still report.
- The skilld install chip sits in the header aside at every width.
- The nav and the directory table drop the flame emoji for a braille mark,
  and the flame animation goes with it.
- Ranks print in stone dots so the rose stays with the motifs.
DESIGN.md keeps motifs out of navigation and gives each surface one rose
element. The header's rose is the logo dot, so the nav mark draws in
stone. A month board also ranks posts older than the spark's seven days,
and seven blank cells beside a zero read as a broken row, so those rows
show no spark and their posts carry the dates.
@harlan-zw harlan-zw added the harlan-agent-review Pull request triage requires an adversarial Review. Adding this label manually always forces one. label Oct 5, 2026
@harlan-zw

Copy link
Copy Markdown
Collaborator Author

Agent note. Checked by hand, since CI cannot cover it:

  • Rendered /skills/trending (month and week) and /skills on a local dev server in headless Chromium, light and dark, at 1280px and 390px. No horizontal scroll at 390px.
  • Data was a local D1 snapshot of production posts, shifted forward about five days so the seven-day sparks have recent posts. /api/feed/trending returned mentionsByDay on all 30 rows; the bars matched the posts' timestamps.
  • Run chips print owner/repo/name refs, such as npx skilld run latent-spaces/brag/brag.
  • Not exercised: the filler rows (the snapshot filled the board with posts), the all-time range, and the empty state with the converge texture.

@harlan-github-agent harlan-github-agent Bot added harlan-agent-review-required Pull request triage requires an adversarial Review for this head commit. and removed harlan-agent-review Pull request triage requires an adversarial Review. Adding this label manually always forces one. labels Oct 5, 2026
@harlan-zw harlan-zw added the harlan-agent-review-required Pull request triage requires an adversarial Review for this head commit. label Oct 5, 2026
@harlan-github-agent harlan-github-agent Bot added the harlan-agent-running An Agent holds a Task on this issue or pull request right now. label Oct 5, 2026
@harlan-github-agent

harlan-github-agent Bot commented Oct 5, 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-05 16:24 UTC.

GitHub merged this pull request.

No material findings were recorded.

The pull request closed.

@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 5, 2026
@harlan-zw
harlan-zw merged commit b17db34 into main Oct 5, 2026
7 checks passed
@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 5, 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