Skip to content

Remove orphaned research probes (pixelscan, creepjs, headless-tells) - #214

Closed
devin-ai-integration[bot] wants to merge 1 commit into
mainfrom
devin/1789965507-remove-orphan-research-probes
Closed

devin-ai-integration[bot] wants to merge 1 commit into
mainfrom
devin/1789965507-remove-orphan-research-probes

Conversation

@devin-ai-integration

Copy link
Copy Markdown
Contributor

What and why

research/pixelscan-probe.ts, research/creepjs-probe.ts, and research/headless-tells.ts are one-off probes referenced only by tsconfig.harness.json (type-check) and absent from research/README.md, which documents a different three-file set. Delete them and their include entries. Dead-code scan finding sfind-acf87e22f2574bc3acbb2a733cec7223.

Checklist

  • bun run release:check passes locally — only build:harness and lint were run (see below).
  • Guard proxy invariant — unrelated (no runtime change).
  • No secret value reaches the model sandbox — unrelated.
  • Dependency pins — unrelated.
  • Public API changes — none.
  • CI job names unchanged.
  • No unit test imports src/worker.ts.
  • User-visible behaviour — none; research/ is unsupported and not shipped.
  • Nothing private is being committed.

How it was verified

bun run build:harness and bun run lint pass after removal. rg -i 'pixelscan-probe|creepjs-probe|headless-tells' returns no remaining references outside the deleted files.

Link to Devin session: https://app.devin.ai/sessions/19c5905cfa664b2b950bc667226c2501
Open in Devin Desktop: https://app.devin.ai/desktop/session/19c5905cfa664b2b950bc667226c2501?variant=devin
Requested by: @CuriosityOS

Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

I'll fix CI failures and address comments from users with write access. I'll skip comments containing "(aside)".

  • Disable automatic comment, CI, and merge conflict monitoring

@greptile-apps

greptile-apps Bot commented Sep 21, 2026

Copy link
Copy Markdown

RetriggerConfidence Score: 5/5

The PR appears safe to merge because it removes only unreferenced, unsupported research utilities and keeps the harness configuration synchronized with the remaining files.

Summary

This PR removes three undocumented, standalone browser-fingerprinting research probes and removes their corresponding entries from the TypeScript harness configuration.

  • Production code, public APIs, dependencies, and runtime behavior are unchanged.
  • No remaining scripts, workflows, imports, or documentation reference the deleted probes.
  • The remaining research and harness file set stays buildable and type-checked.

Reviews (1) · Last reviewed commit: "Remove orphaned research probes (pixelsc..."

@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

Starting E2E review of BetterWright/betterwright PR #214 (head 3dbed2f); checking PR state and prior coverage first.

@CuriosityOS

Copy link
Copy Markdown
Collaborator

Landed on main via #224 (consolidated dead-code cleanup).

@CuriosityOS
CuriosityOS deleted the devin/1789965507-remove-orphan-research-probes branch September 21, 2026 10:14
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