Skip to content

Commit d7ccd1f

Browse files
jdaltonjdalton
committed
chore(deps): bump catalog + oxfmt sweep
oxlint 1.63.0, oxfmt 0.48.0, taze 19.11.0, ts-preview 20260510.1 (all soak-cleared). Cascade from socket-wheelhouse bbea704. Note: 17 lint errors surfaced by the new oxlint enforcing fleet rules (max-file-lines, no-console-prefer-logger, no-fetch-prefer-http-request, no-logger-newline-literal). Real code-quality drift to address in follow-up commits; this commit captures the bump + format work only.
1 parent 9e9c399 commit d7ccd1f

391 files changed

Lines changed: 3197 additions & 3056 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

‎.claude/agents/code-reviewer.md‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@ Apply the rules from CLAUDE.md sections listed below. Reference the full section
2525
**Test Style**: Functional tests over source scanning. Never read source files and assert on contents. Verify behavior with real function calls.
2626

2727
For each file reviewed, report:
28+
2829
- **Style violations** with file:line
2930
- **Logic issues** (bugs, edge cases, missing error handling)
3031
- **Test gaps** (untested code paths)

‎.claude/agents/refactor-cleaner.md‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,7 @@ Apply these rules from CLAUDE.md exactly:
2323
5. **Verify nothing broke**: Run `pnpm run check` and `pnpm test` after each phase
2424

2525
**What to look for:**
26+
2627
- Unused exports (exported but never imported elsewhere)
2728
- Dead imports (imported but never used)
2829
- Unreachable code paths

‎.claude/hooks/check-new-deps/test/extract-deps.test.mts‎

Lines changed: 38 additions & 94 deletions
Original file line numberDiff line numberDiff line change
@@ -39,12 +39,17 @@ function runHook(
3939
})
4040
return {
4141
code: result.status ?? 1,
42-
stdout: typeof result.stdout === 'string' ? result.stdout : result.stdout.toString(),
43-
stderr: typeof result.stderr === 'string' ? result.stderr : result.stderr.toString(),
42+
stdout:
43+
typeof result.stdout === 'string'
44+
? result.stdout
45+
: result.stdout.toString(),
46+
stderr:
47+
typeof result.stderr === 'string'
48+
? result.stderr
49+
: result.stderr.toString(),
4450
}
4551
}
4652

47-
4853
// ============================================================================
4954
// Unit tests: extractNewDeps per ecosystem
5055
// ============================================================================
@@ -53,20 +58,14 @@ describe('extractNewDeps', () => {
5358
// npm
5459
describe('npm', () => {
5560
it('unscoped', () => {
56-
const d = extractNewDeps(
57-
'package.json',
58-
'"lodash": "^4.17.21"',
59-
)
61+
const d = extractNewDeps('package.json', '"lodash": "^4.17.21"')
6062
assert.equal(d.length, 1)
6163
assert.equal(d[0].type, 'npm')
6264
assert.equal(d[0].name, 'lodash')
6365
assert.equal(d[0].namespace, undefined)
6466
})
6567
it('scoped', () => {
66-
const d = extractNewDeps(
67-
'package.json',
68-
'"@types/node": "^20.0.0"',
69-
)
68+
const d = extractNewDeps('package.json', '"@types/node": "^20.0.0"')
7069
assert.equal(d[0].namespace, '@types')
7170
assert.equal(d[0].name, 'node')
7271
})
@@ -78,22 +77,13 @@ describe('extractNewDeps', () => {
7877
assert.equal(d.length, 3)
7978
})
8079
it('ignores node: builtins', () => {
81-
assert.equal(
82-
extractNewDeps('package.json', '"node:fs": "1"').length,
83-
0,
84-
)
80+
assert.equal(extractNewDeps('package.json', '"node:fs": "1"').length, 0)
8581
})
8682
it('ignores relative', () => {
87-
assert.equal(
88-
extractNewDeps('package.json', '"./foo": "1"').length,
89-
0,
90-
)
83+
assert.equal(extractNewDeps('package.json', '"./foo": "1"').length, 0)
9184
})
9285
it('ignores absolute', () => {
93-
assert.equal(
94-
extractNewDeps('package.json', '"/foo": "1"').length,
95-
0,
96-
)
86+
assert.equal(extractNewDeps('package.json', '"/foo": "1"').length, 0)
9787
})
9888
it('ignores capitalized keys', () => {
9989
assert.equal(
@@ -102,10 +92,7 @@ describe('extractNewDeps', () => {
10292
)
10393
})
10494
it('handles workspace protocol', () => {
105-
const d = extractNewDeps(
106-
'package.json',
107-
'"my-lib": "workspace:*"',
108-
)
95+
const d = extractNewDeps('package.json', '"my-lib": "workspace:*"')
10996
assert.equal(d.length, 1)
11097
})
11198
})
@@ -140,18 +127,12 @@ describe('extractNewDeps', () => {
140127
// golang
141128
describe('golang', () => {
142129
it('with namespace', () => {
143-
const d = extractNewDeps(
144-
'go.mod',
145-
'github.com/gin-gonic/gin v1.9.1',
146-
)
130+
const d = extractNewDeps('go.mod', 'github.com/gin-gonic/gin v1.9.1')
147131
assert.equal(d[0].namespace, 'github.com/gin-gonic')
148132
assert.equal(d[0].name, 'gin')
149133
})
150134
it('stdlib extension', () => {
151-
const d = extractNewDeps(
152-
'go.mod',
153-
'golang.org/x/sync v0.7.0',
154-
)
135+
const d = extractNewDeps('go.mod', 'golang.org/x/sync v0.7.0')
155136
assert.equal(d[0].namespace, 'golang.org/x')
156137
assert.equal(d[0].name, 'sync')
157138
})
@@ -160,34 +141,30 @@ describe('extractNewDeps', () => {
160141
// pypi
161142
describe('pypi', () => {
162143
it('requirements.txt', () => {
163-
const d = extractNewDeps(
164-
'requirements.txt',
165-
'flask>=2.0\nrequests==2.31',
166-
)
144+
const d = extractNewDeps('requirements.txt', 'flask>=2.0\nrequests==2.31')
167145
assert.ok(d.some(x => x.name === 'flask'))
168146
assert.ok(d.some(x => x.name === 'requests'))
169147
})
170148
it('pyproject.toml', () => {
171149
assert.ok(
172-
extractNewDeps('pyproject.toml', '"django>=4.2"')
173-
.some(x => x.name === 'django'),
150+
extractNewDeps('pyproject.toml', '"django>=4.2"').some(
151+
x => x.name === 'django',
152+
),
174153
)
175154
})
176155
it('setup.py', () => {
177156
assert.ok(
178-
extractNewDeps('setup.py', '"numpy>=1.24"')
179-
.some(x => x.name === 'numpy'),
157+
extractNewDeps('setup.py', '"numpy>=1.24"').some(
158+
x => x.name === 'numpy',
159+
),
180160
)
181161
})
182162
})
183163

184164
// gem
185165
describe('gem', () => {
186166
it('single-quoted', () => {
187-
assert.equal(
188-
extractNewDeps('Gemfile', "gem 'rails'")[0].name,
189-
'rails',
190-
)
167+
assert.equal(extractNewDeps('Gemfile', "gem 'rails'")[0].name, 'rails')
191168
})
192169
it('double-quoted with version', () => {
193170
assert.equal(
@@ -259,10 +236,7 @@ describe('extractNewDeps', () => {
259236
// composer
260237
describe('composer', () => {
261238
it('vendor/package', () => {
262-
const d = extractNewDeps(
263-
'composer.json',
264-
'"monolog/monolog": "^3.0"',
265-
)
239+
const d = extractNewDeps('composer.json', '"monolog/monolog": "^3.0"')
266240
assert.equal(d[0].namespace, 'monolog')
267241
assert.equal(d[0].name, 'monolog')
268242
})
@@ -345,9 +319,7 @@ describe('extractNewDeps', () => {
345319
// terraform
346320
describe('terraform', () => {
347321
it('registry module source', () => {
348-
const d = extractTerraform(
349-
'source = "hashicorp/consul/aws"',
350-
)
322+
const d = extractTerraform('source = "hashicorp/consul/aws"')
351323
assert.equal(d[0].type, 'terraform')
352324
assert.equal(d[0].namespace, 'hashicorp')
353325
assert.equal(d[0].name, 'consul')
@@ -365,9 +337,7 @@ describe('extractNewDeps', () => {
365337
// nix flakes
366338
describe('nix flakes', () => {
367339
it('github input', () => {
368-
const d = extractNixFlake(
369-
'inputs.nixpkgs.url = "github:NixOS/nixpkgs"',
370-
)
340+
const d = extractNixFlake('inputs.nixpkgs.url = "github:NixOS/nixpkgs"')
371341
assert.equal(d[0].type, 'github')
372342
assert.equal(d[0].namespace, 'NixOS')
373343
assert.equal(d[0].name, 'nixpkgs')
@@ -394,10 +364,7 @@ describe('extractNewDeps', () => {
394364
assert.equal(d[0].name, 'firefox')
395365
})
396366
it('via extractNewDeps', () => {
397-
const d = extractNewDeps(
398-
'Brewfile',
399-
'brew "wget"\ncask "iterm2"',
400-
)
367+
const d = extractNewDeps('Brewfile', 'brew "wget"\ncask "iterm2"')
401368
assert.equal(d.length, 2)
402369
})
403370
})
@@ -418,10 +385,7 @@ describe('extractNewDeps', () => {
418385
assert.ok(d.some(x => x.name === 'lodash'))
419386
})
420387
it('yarn.lock', () => {
421-
const d = extractNewDeps(
422-
'yarn.lock',
423-
'"lodash@^4.17.21":\n version:',
424-
)
388+
const d = extractNewDeps('yarn.lock', '"lodash@^4.17.21":\n version:')
425389
assert.ok(d.some(x => x.name === 'lodash'))
426390
})
427391
it('Cargo.lock', () => {
@@ -448,10 +412,7 @@ describe('extractNewDeps', () => {
448412
assert.ok(d.some(x => x.name === 'rails'))
449413
})
450414
it('composer.lock', () => {
451-
const d = extractNewDeps(
452-
'composer.lock',
453-
'"name": "monolog/monolog"',
454-
)
415+
const d = extractNewDeps('composer.lock', '"name": "monolog/monolog"')
455416
assert.equal(d[0].namespace, 'monolog')
456417
assert.equal(d[0].name, 'monolog')
457418
})
@@ -490,33 +451,21 @@ describe('extractNewDeps', () => {
490451
assert.equal(d[0].name, 'checkout')
491452
})
492453
it('handles backslash in Cargo.toml path', () => {
493-
const d = extractNewDeps(
494-
'src\\parser\\Cargo.toml',
495-
'serde = "1.0"',
496-
)
454+
const d = extractNewDeps('src\\parser\\Cargo.toml', 'serde = "1.0"')
497455
assert.equal(d.length, 1)
498456
})
499457
})
500458

501459
// pass-through
502460
describe('unsupported files', () => {
503461
it('returns empty for .rs', () => {
504-
assert.equal(
505-
extractNewDeps('main.rs', 'fn main(){}').length,
506-
0,
507-
)
462+
assert.equal(extractNewDeps('main.rs', 'fn main(){}').length, 0)
508463
})
509464
it('returns empty for .js', () => {
510-
assert.equal(
511-
extractNewDeps('index.js', 'x').length,
512-
0,
513-
)
465+
assert.equal(extractNewDeps('index.js', 'x').length, 0)
514466
})
515467
it('returns empty for .md', () => {
516-
assert.equal(
517-
extractNewDeps('README.md', '# hi').length,
518-
0,
519-
)
468+
assert.equal(extractNewDeps('README.md', '# hi').length, 0)
520469
})
521470
})
522471
})
@@ -652,7 +601,8 @@ describe('hook integration', () => {
652601
it('allows nuget package', async () => {
653602
const r = await runHook({
654603
file_path: '/tmp/test.csproj',
655-
new_string: '<PackageReference Include="Newtonsoft.Json" Version="13.0" />',
604+
new_string:
605+
'<PackageReference Include="Newtonsoft.Json" Version="13.0" />',
656606
})
657607
assert.equal(r.code, 0)
658608
})
@@ -673,10 +623,7 @@ describe('hook integration', () => {
673623
assert.equal(r.code, 0)
674624
})
675625
it('passes non-Edit tools', async () => {
676-
const r = await runHook(
677-
{ file_path: '/tmp/package.json' },
678-
'Read',
679-
)
626+
const r = await runHook({ file_path: '/tmp/package.json' }, 'Read')
680627
assert.equal(r.code, 0)
681628
})
682629

@@ -706,10 +653,7 @@ describe('hook integration', () => {
706653
new_string: '"express": "^4", "lodash": "^4", "debug": "^4"',
707654
})
708655
assert.equal(r.code, 0)
709-
assert.ok(
710-
Date.now() - start < 5000,
711-
'batch should be fast',
712-
)
656+
assert.ok(Date.now() - start < 5000, 'batch should be fast')
713657
})
714658

715659
// Write tool

0 commit comments

Comments
 (0)