Skip to content

docs(engine): rename gittensory prose to loopover in packages/loopover-engine (batch A) - #5897

Merged
JSONbored merged 3 commits into
mainfrom
rebrand/prose-engine-a
Jul 15, 2026
Merged

docs(engine): rename gittensory prose to loopover in packages/loopover-engine (batch A)#5897
JSONbored merged 3 commits into
mainfrom
rebrand/prose-engine-a

Conversation

@JSONbored

@JSONbored JSONbored commented Jul 14, 2026

Copy link
Copy Markdown
Owner

Summary

  • Part of the gittensory→loopover rebrand (Epic: complete the gittensory -> loopover rebrand (full cutover, no dual-support) #5705), phase 8b: rename brand-name prose in comments across 19 packages/loopover-engine/src/**/*.ts files (advisory, duplicate detection, focus-manifest, index, issue-rag-query, local-scorer, miner-prediction-metrics, the miner subsystem, prompt-packet, and review content-lane/guardrail modules), plus src/rules/advisory.ts. Comment-only, except:
    • local-scorer.ts's activeModel: "gittensory-deterministic" value — an internal, non-persisted string with no external consumer besides its own test, updated in the same commit.
    • src/rules/advisory.ts and packages/loopover-engine/src/advisory/gate-advisory.ts share a deliberately-divergent gate-decision marker pair (GATE_DECISION_TWIN_PAIR in scripts/check-engine-parity.ts) that must change together in one PR — both renamed here.
  • Two files in this batch (review/guardrail-config.ts, review/linked-issue-label-propagation.ts) are byte-identical twins of already-renamed host files (src/review/**); this batch picks up main's already-correct state for them via rebase, so scripts/check-engine-parity.ts stays green.

Scope

  • The PR title follows type(scope): short summary Conventional Commit format.
  • This PR is focused and does not mix unrelated backend, UI, MCP, docs, dependency, and deploy changes.
  • This follows CONTRIBUTING.md and does not reintroduce GitHub Pages, VitePress, site/, or CNAME.
  • I linked a currently open issue this PR resolves — N/A, maintainer-authored rebrand-epic cleanup (Epic: complete the gittensory -> loopover rebrand (full cutover, no dual-support) #5705), no linked-issue gate applies to owner PRs.

Validation

  • git diff --check
  • npx tsx scripts/check-engine-parity.ts — ok (verified with a real base-ref, matching CI)
  • npm run typecheck — clean

Safety

  • No secrets, wallet details, hotkeys, coldkeys, user PATs, private keys, raw trust scores, private rankings, or private maintainer evidence are exposed.
  • Public GitHub text stays sanitized, low-noise, and does not imply compensation guarantees or optimization tactics.
  • Auth, cookie, CORS, GitHub App, Cloudflare, or session changes include negative-path tests. — N/A, no such changes.
  • API/OpenAPI/MCP behavior is updated and tested where needed. — N/A, comment-only, no behavior change.
  • UI changes use live API data or real empty/error/loading states. — N/A, no UI changes.
  • Visible UI changes include a UI Evidence section. — N/A, no UI changes.
  • Public docs/changelogs are updated where needed; changelogs are only edited for release-prep PRs (none touched here).

Notes

@superagent-security

Copy link
Copy Markdown
Contributor

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

@loopover-orb loopover-orb Bot added the gittensor:bug Gittensor-scored bug fix — scores a 0.05x multiplier. label Jul 14, 2026
@loopover-orb

loopover-orb Bot commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Caution

🟥🟥🟥🟥🟥🟥🟥🟥🟥🟥🟥🟥

🛑 LoopOver review result - fixes required

Review updated: 2026-07-15 00:02:01 UTC

20 files · 1 AI reviewer · 1 blocker · readiness 95/100 · CI failing · blocked

🛑 Suggested Action - Manual Review

Review summary
Comment-only rebrand rename of gittensory→loopover prose across 19 files in packages/loopover-engine, plus one internal test-only string constant (`activeModel`) updated consistently in both source and its test. The change is mechanical, low-risk, and self-contained; no behavioral or API surface changes are introduced, and the string-literal update is correctly paired with its corresponding test assertion update.

Nits — 3 non-blocking
  • local-scorer.ts:59 the `activeModel` string change is a value, not prose — worth flagging explicitly in the PR description as intentional (which it already is) so future rebrand batches don't assume this file is comment-only by pattern-matching the title.
  • Given this is a maintainer-authored, mechanical, no-linked-issue cleanup PR per the epic (Epic: complete the gittensory -> loopover rebrand (full cutover, no dual-support) #5705), consider batching more files per PR if `check-engine-parity.ts` and typecheck stay green, to reduce total PR count for this rebrand.
  • Code changes lack test evidence — Add focused regression tests or explain why existing coverage is sufficient.

Why this is blocked

  • No linked issue detected — If this PR is intended to solve an issue, link it explicitly in the PR body.
📋 Copy for AI agents — paste into your coding agent
Fix the following blocker(s) from this PR review:

1. No linked issue detected — If this PR is intended to solve an issue, link it explicitly in the PR body.

CI checks failing

  • validate
  • validate-tests (6)
Signal Result Evidence
Code review ❌ 1 blocker 1 reviewer
Linked issue ✅ No-issue rationale PR body explains why no issue is linked.
Related work ✅ No active overlap found No same-issue or scoped active PR overlap found.
Change scope ✅ 20/20 Low review scope from cached public metadata (no linked issue context).
Validation posture ✅ 25/25 PR body includes validation/test evidence.
Contributor workload ✅ 10/10 Author activity: 43 registered-repo PR(s), 35 merged, 373 issue(s).
Contributor context ✅ Confirmed Gittensor contributor JSONbored; Gittensor profile; 43 PR(s), 373 issue(s).
Gate result ❌ Blocking Repo-configured hard blocker found.
Improvement ⚠️ ℹ️ None detected risk: low · value: none — No structural-improvement signals were detected for this PR.
Review context
  • Author: JSONbored
  • Role context: owner (maintainer lane)
  • Public audience mode: oss maintainer
  • Lane context: Repository registration is not available in the local Gittensory cache.
  • Public profile languages: not available
  • Official Gittensor activity: 43 PR(s), 373 issue(s).
  • PR-specific overlap: none found.
Contributor next steps
  • Treat this as maintainer-lane context rather than normal contributor-lane activity.
  • Triage stale or unlinked PRs.
  • No action.
Signal definitions
  • Related work = same linked issue, overlapping active PRs, or title/path similarity.
  • Change scope = cached public metadata such as size labels, draft state, and review-burden hints.
  • Validation posture = whether the PR provides enough public validation/test evidence for maintainer review.
  • Contributor workload = public contributor activity and cleanup pressure, not a repo-wide quality failure.
  • Contributor context = public GitHub/Gittensor identity context; non-Gittensor status is not a blocker.
[BETA] Chat with Gittensory

Ask Gittensory a question about this PR directly in a comment — grounded only in the same cached, public-safe facts shown above, never a new claim.

  • @gittensory ask <question> answers contribution-quality Q&A with source citations and freshness.
  • @gittensory chat <question> answers in natural prose from cached decision-pack facts via local inference (maintainer/collaborator; read-only).
  • A plain-language @gittensory mention with a real question is routed to the closest matching read-only command automatically -- no exact syntax required.

Full command reference: https://gittensory.aethereal.dev/docs/gittensory-commands

🟩 Safe / merged · 🟦 Advisory · 🟨 Held for review · 🟥 Blocked / closed


💰 Earn for open-source contributions like this. Gittensor lets GitHub contributors earn for the work they already do — register to start earning →.

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

  • Re-run LoopOver review

@loopover-orb loopover-orb Bot added the manual-review Gittensor contributor context label Jul 14, 2026
JSONbored added a commit that referenced this pull request Jul 14, 2026
- .loopover.yml: apply the same prose rename as its required-identical
  twin src/config/gittensory-repo-focus-manifest.ts's bundled YAML
  (test/unit/gittensory-focus-manifest.test.ts checks byte-identity).
- test/unit/e2e-test-commit.test.ts: update two assertions to match
  the renamed e2e/loopover-pr-N.spec.ts path (src/github/e2e-test-commit.ts
  is in this batch).
- src/rules/advisory.ts: revert the prose rename here -- it shares a
  gate-decision marker pair with packages/loopover-engine/src/advisory/
  gate-advisory.ts (already renamed in #5897) and scripts/check-engine-parity.ts
  requires both sides change together; #5897 will own this pair instead.
@JSONbored
JSONbored force-pushed the rebrand/prose-engine-a branch from 7db7860 to 3994bca Compare July 14, 2026 22:27
JSONbored added a commit that referenced this pull request Jul 14, 2026
- .loopover.yml: apply the same prose rename as its required-identical
  twin src/config/gittensory-repo-focus-manifest.ts's bundled YAML
  (test/unit/gittensory-focus-manifest.test.ts checks byte-identity).
- test/unit/e2e-test-commit.test.ts: update two assertions to match
  the renamed e2e/loopover-pr-N.spec.ts path (src/github/e2e-test-commit.ts
  is in this batch).
- src/rules/advisory.ts: revert the prose rename here -- it shares a
  gate-decision marker pair with packages/loopover-engine/src/advisory/
  gate-advisory.ts (already renamed in #5897) and scripts/check-engine-parity.ts
  requires both sides change together; #5897 will own this pair instead.
@JSONbored
JSONbored force-pushed the rebrand/prose-engine-a branch from 3994bca to b259da1 Compare July 14, 2026 22:32
JSONbored added a commit that referenced this pull request Jul 14, 2026
- .loopover.yml: apply the same prose rename as its required-identical
  twin src/config/gittensory-repo-focus-manifest.ts's bundled YAML
  (test/unit/gittensory-focus-manifest.test.ts checks byte-identity).
- test/unit/e2e-test-commit.test.ts: update two assertions to match
  the renamed e2e/loopover-pr-N.spec.ts path (src/github/e2e-test-commit.ts
  is in this batch).
- src/rules/advisory.ts: revert the prose rename here -- it shares a
  gate-decision marker pair with packages/loopover-engine/src/advisory/
  gate-advisory.ts (already renamed in #5897) and scripts/check-engine-parity.ts
  requires both sides change together; #5897 will own this pair instead.
@JSONbored
JSONbored force-pushed the rebrand/prose-engine-a branch from b259da1 to f3f84eb Compare July 14, 2026 23:02
JSONbored added a commit that referenced this pull request Jul 14, 2026
- .loopover.yml: apply the same prose rename as its required-identical
  twin src/config/gittensory-repo-focus-manifest.ts's bundled YAML
  (test/unit/gittensory-focus-manifest.test.ts checks byte-identity).
- test/unit/e2e-test-commit.test.ts: update two assertions to match
  the renamed e2e/loopover-pr-N.spec.ts path (src/github/e2e-test-commit.ts
  is in this batch).
- src/rules/advisory.ts: revert the prose rename here -- it shares a
  gate-decision marker pair with packages/loopover-engine/src/advisory/
  gate-advisory.ts (already renamed in #5897) and scripts/check-engine-parity.ts
  requires both sides change together; #5897 will own this pair instead.
@JSONbored
JSONbored force-pushed the rebrand/prose-engine-a branch from f3f84eb to 3b7247f Compare July 14, 2026 23:18
JSONbored added a commit that referenced this pull request Jul 14, 2026
- .loopover.yml: apply the same prose rename as its required-identical
  twin src/config/gittensory-repo-focus-manifest.ts's bundled YAML
  (test/unit/gittensory-focus-manifest.test.ts checks byte-identity).
- test/unit/e2e-test-commit.test.ts: update two assertions to match
  the renamed e2e/loopover-pr-N.spec.ts path (src/github/e2e-test-commit.ts
  is in this batch).
- src/rules/advisory.ts: revert the prose rename here -- it shares a
  gate-decision marker pair with packages/loopover-engine/src/advisory/
  gate-advisory.ts (already renamed in #5897) and scripts/check-engine-parity.ts
  requires both sides change together; #5897 will own this pair instead.
@JSONbored
JSONbored force-pushed the rebrand/prose-engine-a branch from 3b7247f to 61b81d5 Compare July 14, 2026 23:33
JSONbored added a commit that referenced this pull request Jul 14, 2026
- .loopover.yml: apply the same prose rename as its required-identical
  twin src/config/gittensory-repo-focus-manifest.ts's bundled YAML
  (test/unit/gittensory-focus-manifest.test.ts checks byte-identity).
- test/unit/e2e-test-commit.test.ts: update two assertions to match
  the renamed e2e/loopover-pr-N.spec.ts path (src/github/e2e-test-commit.ts
  is in this batch).
- src/rules/advisory.ts: revert the prose rename here -- it shares a
  gate-decision marker pair with packages/loopover-engine/src/advisory/
  gate-advisory.ts (already renamed in #5897) and scripts/check-engine-parity.ts
  requires both sides change together; #5897 will own this pair instead.
JSONbored added a commit that referenced this pull request Jul 14, 2026
…irs (#5895)

* docs(core): rename gittensory prose to loopover in remaining src subdirs

Rebrand cutover cleanup: update brand-name prose in comments across
31 files in src/api, src/auth, src/config, src/db, src/env.d.ts,
src/github, src/index.ts, src/integrations, src/notifications,
src/openapi, src/orb, src/queue, src/rules, src/scoring, src/server.ts,
src/settings, src/types.ts, and src/upstream. Comment-only, no
behavior change. Includes the matching engine-twin fix for
src/settings/autonomy.ts (packages/loopover-engine/src/settings/autonomy.ts)
to keep scripts/check-engine-parity.ts passing.

Deliberate legacy references left untouched: GITTENSORY_LEGACY_*
check-run name constants, GITTENSORY-[A-Z0-9]+ Sentry issue slugs,
the gittensory.aethereal.dev/gittensory-api.aethereal.dev domain
aliases, gittensory-selfhost container-image tag, and the
gittensory-repo-focus-manifest module path (file itself not yet
renamed).

* fix(core): resync twin files and stale test fixtures broken by rename

- .loopover.yml: apply the same prose rename as its required-identical
  twin src/config/gittensory-repo-focus-manifest.ts's bundled YAML
  (test/unit/gittensory-focus-manifest.test.ts checks byte-identity).
- test/unit/e2e-test-commit.test.ts: update two assertions to match
  the renamed e2e/loopover-pr-N.spec.ts path (src/github/e2e-test-commit.ts
  is in this batch).
- src/rules/advisory.ts: revert the prose rename here -- it shares a
  gate-decision marker pair with packages/loopover-engine/src/advisory/
  gate-advisory.ts (already renamed in #5897) and scripts/check-engine-parity.ts
  requires both sides change together; #5897 will own this pair instead.

* chore: regenerate openapi.json after JSONbored/gittensory example rename
…r-engine (batch A)

Rebrand cutover cleanup: update brand-name prose in comments across
19 packages/loopover-engine/src/**/*.ts files (advisory, duplicate
detection, focus-manifest, index, issue-rag-query, local-scorer,
miner-prediction-metrics, the miner subsystem, prompt-packet, and
review content-lane/guardrail modules). Comment-only, except the
'gittensory-deterministic' activeModel value in local-scorer.ts
(no external consumer -- test/local-scorer.test.ts updated in the
same commit).
…rose

Completes the gittensory->loopover rename for the deliberately-
divergent gate-decision twin (packages/loopover-engine/src/advisory/
gate-advisory.ts, already renamed) -- scripts/check-engine-parity.ts
requires both sides of this pair change together in one PR.
…corer.test.ts

src/signals/local-scorer.ts is a thin re-export shim over
packages/loopover-engine/src/local-scorer.ts (renamed in this batch),
so this root-level test's hardcoded 'gittensory-deterministic'
expectation broke at runtime even though it never imports the engine
package directly. Missed earlier because it's a different file from
packages/loopover-engine/test/local-scorer.test.ts, already fixed.
@JSONbored
JSONbored force-pushed the rebrand/prose-engine-a branch from 596fe2d to 16d09f4 Compare July 15, 2026 00:03
@JSONbored
JSONbored merged commit e26b263 into main Jul 15, 2026
16 checks passed
@JSONbored
JSONbored deleted the rebrand/prose-engine-a branch July 15, 2026 00:08
This was referenced Jul 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gittensor:bug Gittensor-scored bug fix — scores a 0.05x multiplier. manual-review Gittensor contributor context

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant