Skip to content

fix(ios): write the download-lock separator as \0 instead of a raw NUL byte - #143

Merged
V3RON merged 2 commits into
mainfrom
task/141
Sep 10, 2026
Merged

fix(ios): write the download-lock separator as \0 instead of a raw NUL byte#143
V3RON merged 2 commits into
mainfrom
task/141

Conversation

@V3RON

@V3RON V3RON commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Closes #141

src/drivers/ios/index.ts:594 held a literal NUL character in the source text. That one byte made grep and ripgrep classify the whole 2011-line iOS driver as binary, so the file was invisible to a normal search. The \0 escape produces the same runtime string, so the download-lock key is unchanged.

The guard test from triage comes with it, enumerating git ls-files so it covers every tracked file rather than one directory.

  • The source byte is gone. readFileSync("src/drivers/ios/index.ts").indexOf(0) returns -1; it was 27650.
  • The file is searchable again. rg -c advisor src/drivers/ios/index.ts prints 2 instead of binary file matches.
  • Behaviour is unchanged. iOS driver suite: 115 pass, 1 skipped.
  • The regression test fails for the right reason. Reverting the one byte turns tracked files > contain no raw NUL byte, so grep and ripgrep can search them red with src/drivers/ios/index.ts:594 (byte 27650).
  • pnpm check. Typecheck, oxlint, oxfmt and the unit suite are clean. Two e2e files failed on a stray-daemon teardown under full-suite load and pass in isolation on this branch; the same files pass on main in isolation too.

Written by an agent.


Generated by Claude Code

V3RON and others added 2 commits September 10, 2026 12:25
…L byte

The literal NUL in the source made grep and ripgrep classify the whole
2011-line iOS driver as binary, so the file was invisible to a normal
search. The escape produces the same runtime string.

Closes #141

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_018HHP7Th9NqWfhexmp4V9Pf
@V3RON
V3RON merged commit 681baa4 into main Sep 10, 2026
14 checks passed
@V3RON
V3RON deleted the task/141 branch September 10, 2026 16:42
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.

src/drivers/ios/index.ts contains a raw NUL byte, so grep and ripgrep treat it as binary

2 participants