Commit f5d7c6e

committed

docs(standards): add process.chdir() prohibition to code style

Added working directory rule to CLAUDE.md: - Never use process.chdir() as it breaks tests, worker threads, and causes race conditions - Always use { cwd } options and absolute paths instead - Pass { cwd: absolutePath } to spawn/exec/fs operations This rule prevents global state mutation anti-pattern that causes test failures in worker threads.

1 parent 25021aa commit f5d7c6eCopy full SHA for f5d7c6e

File tree

1 file changed

-0

lines changed

CLAUDE.md

1 file changed

-0

lines changed

`‎CLAUDE.md‎`

Lines changed: 5 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -146,6 +146,11 @@ TypeScript implementation of [Package URL specification](https://github.com/pack`
`146`	`146`	`import type { ArgParserConfig } from './argv.mjs'`
`147`	`147`	- ❌ `import { parseScriptArgs, isQuiet, type ArgParserConfig } from './argv.mjs'`
`148`	`148`
	`149`	`+#### Working Directory`
	`150`	+- 🚨 NEVER use `process.chdir()` - use `{ cwd }` options and absolute paths instead
	`151`	`+ - Breaks tests, worker threads, and causes race conditions`
	`152`	+ - Always pass `{ cwd: absolutePath }` to spawn/exec/fs operations
	`153`	`+`
`149`	`154`	`#### Optional Properties`
`150`	`155`	With `exactOptionalPropertyTypes`, assign conditionally:
`151`	`156`	- ✅ `if (value !== undefined) { this.prop = value }`

0 commit comments

Comments

(0)

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit f5d7c6e

File tree

1 file changed

1 file changed

`‎CLAUDE.md‎`

0 commit comments