test(miner): rename gittensory-miner/ams prose to loopover-miner/ams - #5874
Merged
Conversation
Rebrand cutover cleanup: update describe()/it() titles, mkdtempSync tmpdir prefixes, and comments across 26 test/unit files to use the current loopover-miner/loopover-ams naming. Also fixes a stale "gittensory-miner benchmark" banner string in benchmark.mjs and a check-miner-package.test.ts fixture filename that no longer matched the real package-name prefix it was meant to test.
Contributor
|
Superagent didn't find any vulnerabilities or security issues in this PR. |
24 tasks
18 tasks
Closed
24 tasks
This was referenced Jul 14, 2026
Closed
Closed
Closed
Closed
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
test/unit/*.tsfiles whosedescribe()/it()titles,mkdtempSync/tempRoottmpdir prefixes, and inline comments still used the baregittensory-miner/gittensory-amsstrings as human-readable labels.packages/loopover-miner/scripts/benchmark.mjs's printed banner still said"gittensory-miner benchmark", andcheck-miner-package.test.ts's "rejects an unexpected miner bin that matches the package name prefix" test used abin/gittensory-miner-backdoor.jsfixture filename that no longer actually matches the real package prefix (loopover-miner), so it wasn't testing what its title claimed.Scope
type(scope): short summaryConventional Commit format.CONTRIBUTING.mdand does not reintroduce GitHub Pages, VitePress,site/, orCNAME.Validation
git diff --checknpm run actionlint(vianpm run test:ci)npm run typecheck(vianpm run test:ci)npm run test:coverage— full unsharded run, 100% green; test-only diff socodecov/patchhas no measurablesrc/**lines beyond the existing-covered benchmark.mjs linenpm run test:workers(vianpm run test:ci)npm run build:mcp(vianpm run test:ci)npm run test:mcp-pack(vianpm run test:ci)npm run ui:openapi:check(vianpm run test:ci)npm run ui:lint(vianpm run test:ci)npm run ui:typecheck(vianpm run test:ci)npm run ui:build(vianpm run test:ci)npm audit --audit-level=moderatenpm run test:cigate run locally, exit 0Safety
UI Evidencesection. — N/A, no UI changes.Notes
describe()/tmpdir-prefix renames tracked in rebrand epic Epic: complete the gittensory -> loopover rebrand (full cutover, no dual-support) #5705.