chore(config): fix stale gittensory prose in infra/root config files#6841
Merged
Conversation
Sweeps the remaining cosmetic "gittensory" references in root and observability config files where the underlying identifier has already moved to "loopover" -- .dockerignore's loopover-config mount exclusion (was excluding the old, no-longer-existing gittensory-config path), the /loopover/shot route and LoopoverPostgresConnectionPressure alert name in .env.example, the loopover-app.env setup-wizard default, plus assorted self-referential prose in .env.example/.gitleaks.toml/ .loopover-miner.yml.example/orb-manifest.json/vitest.config.ts/ loki/promtail configs. Also renames the stale gittensory-engine-scaffold.test.ts to loopover-engine-scaffold.test.ts (it already tests the real @loopover/engine package name). Leaves every dead GITTENSORY_* env var name in .env.example untouched (each is documented as "no longer read" -- the historical name is the point) and leaves the SENTRY_RELEASE/LOOPOVER_VERSION example alone, adding a short pointer to why it still says gittensory-selfhost (that default is deliberately pinned by scripts/deploy-selfhost-prebuilt.sh and test/unit/selfhost-sentry-release.test.ts).
Contributor
|
Superagent didn't find any vulnerabilities or security issues in this PR. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #6841 +/- ##
=======================================
Coverage 93.63% 93.63%
=======================================
Files 683 683
Lines 68154 68154
Branches 18704 18704
=======================================
Hits 63818 63818
Misses 3350 3350
Partials 986 986
Flags with carried forward coverage won't be shown. Click here to find out more. |
This was referenced Jul 17, 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
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.
Summary
.dockerignorewas excluding a build-context path namedgittensory-config, which no longer exists --docker-compose.ymlmounts the operator's private per-repo config from./loopover-configtoday. Fixed the exclusion to match (this one has a small privacy angle: that directory holds runtime-only private review config that must never enter image layers per the file's own comment)..env.example: the documented/gittensory/shotroute (real route is/loopover/shot, persrc/review/visual/shot.ts), theGittensoryPostgresConnectionPressurealert name (real alert isLoopoverPostgresConnectionPressure, perprometheus/rules/alerts.yml), and theSETUP_OUTPUT_PATHdefault (src/server.tsfalls back to/data/loopover-app.env), plus assorted self-referential prose ("gittensory's central collector" etc.) and illustrative example values (MINIO_BUCKET,DOMAIN)..gitleaks.toml,.loopover-miner.yml.example,orb-manifest.json,vitest.config.ts,loki/loki-config.yml,promtail/promtail-config.yml: same class of stale self-referential prose/title fixes.docker-compose.yml: a stray#gittensory-4327issue-comment prefix -- confirmed issue GPU passthrough for Ollama + embedding benchmark vs. the current CPU-bound setup #4327 exists in this repo and no other file uses a#<reponame>-NNNNconvention, so this was just a leftover repo-name prefix, not a real convention.test/unit/gittensory-engine-scaffold.test.ts->test/unit/loopover-engine-scaffold.test.ts(including itsdescribe()title) -- it already asserts the real, current@loopover/enginepackage identity; only the filename/title were stale. Test files aren't referenced by path elsewhere, so this rename has no other blast radius (verified with a repo-wide grep).GITTENSORY_*env var name in.env.exampleuntouched (each is explicitly commented "no longer read" -- the whole point of those lines is to document the old name for anyone migrating a stale deployment) and left theSENTRY_RELEASE/LOOPOVER_VERSIONself-host example alone (itsgittensory-selfhost@<version>value is intentionally pinned byscripts/deploy-selfhost-prebuilt.sh's actual default andtest/unit/selfhost-sentry-release.test.ts) -- added a short inline pointer to why, so this doesn't get "fixed" again by mistake later.Scope
type(scope): short summaryConventional Commit format.CONTRIBUTING.mdand does not reintroduce GitHub Pages, VitePress,site/, orCNAME.Validation
git diff --checknpm run actionlintnpm run typechecknpm run test:coveragelocally (nosrc/**lines changed; config/docs/test-rename only)npm run test:workersnpm run build:mcpnpm run test:mcp-packnpm run ui:openapi:checknpm run ui:lintnpm run ui:typechecknpm run ui:buildnpm audit --audit-level=moderate(0 vulnerabilities)npm run test:cigreen end to end, captured exit code 0 directly (not through a pipe) to be certaindocker compose config --quietpasses against the editeddocker-compose.ymlSafety
UI Evidencesection -- N/A, no visible/rendered change.Notes
gittensoryreferences left over from the rebrand, split by area (small, focused PRs per repo convention). This one covers root/infra config files; docs(selfhost): fix stale gittensory-selfhost self-host doc defaults #6826 covered self-hosting docs.wrangler.jsonc(a handful of similar self-referential comment fixes) is deliberately being handled as its own, even-more-isolated follow-up PR given its stakes as the live Cloudflare Worker config.