Skip to content

feat(comments): Gittensor viral loop — earn footer + minimal invite + AI assessment - #620

Merged
JSONbored merged 2 commits into
mainfrom
feat/gittensory-viral-footer
Jun 12, 2026
Merged

feat(comments): Gittensor viral loop — earn footer + minimal invite + AI assessment#620
JSONbored merged 2 commits into
mainfrom
feat/gittensory-viral-footer

Conversation

@JSONbored

@JSONbored JSONbored commented Jun 12, 2026

Copy link
Copy Markdown
Owner

Three cohesive upgrades to the gittensory PR comment that serve the Gittensor growth mandate (more outside contributors + more registrations via our repos). All shipped together; signed commits.

1. Always-on earn footer

Every reviewed PR carries a permanent "Earn for open-source contributions like this → register to start earning" CTA (the link persists forever — free marketing on every PR). Points at the repo's public Gittensor miner page (social proof). Used by the PR panel, command responses, and the closed-PR notice.

2. Minimal invite for non-registered contributors

A contributor with no Gittensor footprint (detection.detected === false) gets a brief welcome + earn invite instead of the full mining-readiness panel (which doesn't apply to them). Known contributors (official or cached) still get the full review. The earn footer appears either way.

3. Combined AI assessment (enabled)

For known contributors, the panel now includes a 🤖 AI assessment — one short, public-safe note useful to both the contributor (how to get this PR reviewed/accepted faster) and the maintainer (what it does + any risk). Reuses the existing ai-summaries layer: daily neuron budget, public-forbidden-word sanitizer, and a guaranteed fail-safe (the deterministic panel stands alone if AI is disabled/over budget/unsafe). Enables AI_SUMMARIES_ENABLED + AI_PUBLIC_COMMENTS_ENABLED.

Safety / quality

  • Public comments use only "earn" wording — never reward/payout/score (those throw in sanitizePublicComment); unit tests guard both the footer and the AI output.
  • Full suite green (1516 passed). All commits signed/verified.

⚠️ Note: merging this turns on live Workers-AI spend per detected-contributor PR (capped by AI_DAILY_NEURON_BUDGET=10000/day).

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jun 12, 2026

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
gittensory-ui cdd081d Commit Preview URL

Branch Preview URL
Jun 12 2026, 02:24 PM

@ghost

ghost commented Jun 12, 2026

Copy link
Copy Markdown

gittensory · advisory review

Reviewed 8 changed file(s) — two independent AI reviewers.

Suggested action:Safe to merge — both reviewers found no blocking issues.

Reviewer A · gpt-oss-120b — recommends ✅ merge
This PR introduces a shared footer with an earn CTA for public Gittensory comments, integrates it across comment generation, adds a minimal invite comment for non‑registered contributors, and expands test coverage. The changes compile cleanly and respect existing sanitisation rules.

Suggestions

  • Consider extending the detection type definition to include the optional source field for better type safety.
  • Update any snapshot‑style tests that assert exact comment bodies to account for the new multi‑line footer.
  • Add a short comment in the code explaining why the early return for non‑detected contributors is safe.

Worth double-checking

  • Existing tests that check exact comment strings may fail due to the added footer lines.
  • If sanitizePublicComment ever adds new forbidden words, the footer may need updating.

Reviewer B · nemotron-3-120b-a12b — recommends ✅ merge
The PR adds a new 'viral loop' feature: a permanent earn CTA footer in all public Gittensory comments, plus a minimal invite comment (without readiness panel) for non-registered contributors. Changes are focused, well-tested, and align with the project's growth goals. The footer avoids forbidden words and uses repo-specific earn URLs for social proof.

Suggestions

  • Consider adding a test for the minimal invite comment's alert block rendering (e.g., verify [!NOTE] is processed correctly by formatAlertBlock to ensure proper GitHub styling).
  • In src/github/footer.ts, the gittensoryFooter function's default param {} could be simplified to opts?: { earnUrl?: string } to avoid unnecessary object creation, though impact is negligible.

Worth double-checking

  • The minimal invite logic (!args.detection.detected) assumes detection objects always have a detected boolean—confirm this holds for all detection types (official/cache/none).
  • Ensure the earn URL encoding in gittensorRepoEarnUrl handles edge cases like repo names with special characters (already uses encodeURIComponent, so low risk).

@ghost ghost added the gittensory-review label Jun 12, 2026
@ghost

ghost commented Jun 12, 2026

Copy link
Copy Markdown

Note

Gittensory Gate skipped

PR closed before full evaluation. No late first comment was created.

Signal Result Evidence Action
Gate result ⚠️ Skipped #620 is no longer open. No action.

Checked by Gittensory, a quiet PR intelligence layer for OSS maintainers.

@ghost ghost added the gittensory:reviewed label Jun 12, 2026
@dosubot dosubot Bot added size:M and removed size:XS labels Jun 12, 2026
@superagent-security

Copy link
Copy Markdown
Contributor

Superagent didn't find any vulnerabilities or security issues in this PR.

@JSONbored
JSONbored force-pushed the feat/gittensory-viral-footer branch from 56ce890 to c570547 Compare June 12, 2026 13:52
@JSONbored JSONbored changed the title feat(comments): audience-aware Gittensor earn footer (viral loop) feat(comments): always-on earn footer + minimal invite for non-registered contributors Jun 12, 2026
@JSONbored JSONbored self-assigned this Jun 12, 2026
Adds a shared `gittensoryFooter()` (src/github/footer.ts) used by the PR panel,
command responses, and the closed-PR notice. It ALWAYS renders an 'earn for your
contributions → register to start earning' CTA — a permanent, free marketing surface
on every reviewed PR (the link persists forever): non-registered authors see the
invite, and viewers of a registered contributor's PR see it too. The CTA points at
the repo's public Gittensor miner page (social proof for that repo).

Uses only 'earn' wording — never reward/payout/score (those throw in
sanitizePublicComment); a unit test guards the invariant. The registered/non-
registered distinction belongs in the review body (full panel vs. minimal), not here.
When a PR author has no Gittensor footprint (not detected via official API or
cache), gittensory's readiness/contribution analysis doesn't apply — so post a
brief welcome + earn invite instead of the full readiness panel. A known
contributor (official or cached) still gets the full review. The always-on earn
footer appears either way. Keyed on detection.detected so cached contributors
keep the full panel. Adds a minimal-invite unit test.
@JSONbored
JSONbored force-pushed the feat/gittensory-viral-footer branch from 90ba298 to cdd081d Compare June 12, 2026 14:21
@JSONbored
JSONbored merged commit 2d9bfda into main Jun 12, 2026
11 checks passed
@github-project-automation github-project-automation Bot moved this from Todo to Done in gittensory - v1 roadmap Jun 12, 2026
@JSONbored
JSONbored deleted the feat/gittensory-viral-footer branch June 12, 2026 14:29
@JSONbored JSONbored changed the title feat(comments): always-on earn footer + minimal invite for non-registered contributors feat(comments): Gittensor viral loop — earn footer + minimal invite + AI assessment Jun 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant