Chris Arderne
c7861be520
Once this is merged, oxlint is at a pretty sensible baseline. **Enable `no-unused-vars`, `typescript/consistent-type-imports`, and `import/no-duplicates` lint rules** Turns on three previously-disabled oxlint rules across the monorepo and fixes all violations: - **`no-unused-vars`** – enabled as an error with standard ignore patterns: unused function arguments are ignored by default (`args: "none"`), variables/caught errors/destructured array elements prefixed with `_` are allowed, and rest siblings are permitted. - **`typescript/consistent-type-imports`** – enforced as an error; all type-only imports now use the `import type` syntax. - **`import/no-duplicates`** – enforced as an error; duplicate import statements from the same module have been merged. The remaining commits clean up the violations found across the codebase: removing unused variables/imports/type aliases, adding `_` prefixes to intentionally unused bindings, fixing duplicate imports, and converting value imports to `import type` where appropriate.
30 lines
820 B
TypeScript
30 lines
820 B
TypeScript
import type { Attributes } from "@opentelemetry/api";
|
|
import { startActiveSpan } from "~/v3/tracer.server";
|
|
|
|
export async function parseRequestJsonAsync(
|
|
request: Request,
|
|
attributes?: Attributes
|
|
): Promise<unknown> {
|
|
return await startActiveSpan(
|
|
"parseRequestJsonAsync()",
|
|
async (span) => {
|
|
span.setAttribute("content-length", parseInt(request.headers.get("content-length") ?? "0"));
|
|
span.setAttribute("content-type", request.headers.get("content-type") ?? "application/json");
|
|
span.setAttribute("experiment.async", false);
|
|
|
|
const rawText = await startActiveSpan("request.text()", async () => {
|
|
return await request.text();
|
|
});
|
|
|
|
if (rawText.length === 0) {
|
|
return;
|
|
}
|
|
|
|
return JSON.parse(rawText);
|
|
},
|
|
{
|
|
attributes,
|
|
}
|
|
);
|
|
}
|