Skip to content

Repository navigation

docs(agents): correct build and http-client facts #692

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
Sarkis Varozian (sarkis) wants to merge 1 commit into from
Closed

docs(agents): correct build and http-client facts #692

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 8 additions & 8 deletions AGENTS.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
# CLAUDE.md

🚨 **MANDATORY**: Act as principal-level engineer with deep expertise in TypeScript, Node.js, and SDK development.
Act as principal-level engineer with deep expertise in TypeScript, Node.js, and SDK development.

<!-- <fleet> -->

Expand Down Expand Up @@ -175,14 +175,14 @@

## 🏗️ SDK-Specific

Socket SDK for JavaScript/TypeScript - programmatic access to Socket.dev security analysis. Build: `pnpm run build` (esbuild → ESM, node18+); test: `pnpm test`; coverage: `pnpm run cover`.
Socket SDK for JavaScript/TypeScript - programmatic access to Socket.dev security analysis. Build: `pnpm run build` (rolldown → CJS, node18+); test: `pnpm test`; coverage: `pnpm run cover`.

- 🚨 Never call `fetch()`; use `createGetRequest` / `createRequestWithJson` from `src/http-client.ts`. [`architecture`](docs/repo/agents.md/architecture.md)
- 🚨 Reach an external URL by passing a different `baseUrl` to `createGetRequest`, never by dropping to `fetch()`. [`architecture`](docs/repo/agents.md/architecture.md)
- 🚨 Source files use the `.mts` extension and carry a mandatory `@file` header. [`architecture`](docs/repo/agents.md/architecture.md)
- 🚨 `"use strict"` is FORBIDDEN in `.mjs`/`.mts` - ES modules are already strict. [`architecture`](docs/repo/agents.md/architecture.md)
- Never call `fetch()`; use `createGetRequest` / `createRequestWithJson` from `src/http-client.mts`. [`architecture`](docs/repo/agents.md/architecture.md)
- Reach an external URL by passing a different `baseUrl` to `createGetRequest`, never by dropping to `fetch()`. [`architecture`](docs/repo/agents.md/architecture.md)
- Source files use the `.mts` extension and carry a `@file` header. [`architecture`](docs/repo/agents.md/architecture.md)
- Omit `"use strict"` in `.mjs`/`.mts` - ES modules are already strict. [`architecture`](docs/repo/agents.md/architecture.md)
- Code is semicolon-free, enforced by oxfmt. [`architecture`](docs/repo/agents.md/architecture.md)
- 🚨 No `any` - use `unknown` or a specific type. [`architecture`](docs/repo/agents.md/architecture.md)
- No `any` - use `unknown` or a specific type. [`architecture`](docs/repo/agents.md/architecture.md)
- `logger.error('')` and `logger.log('')` need the empty-string argument. [`architecture`](docs/repo/agents.md/architecture.md)
- 🚨 Never put `--` before a vitest test path; it runs ALL tests. [`architecture`](docs/repo/agents.md/architecture.md)
- Never put `--` before a vitest test path; it runs every test. [`architecture`](docs/repo/agents.md/architecture.md)
- Full layout, command catalog, config-file table, sorting rules, testing helpers, CI mandate and SDK notes live in the architecture doc. [`architecture`](docs/repo/agents.md/architecture.md)
Loading