Skip to content

add logger to typescript package#332

Merged
xzrderek merged 5 commits intomainfrom
derekx/typescript-package-logger
Nov 17, 2025
Merged

add logger to typescript package#332
xzrderek merged 5 commits intomainfrom
derekx/typescript-package-logger

Conversation

@xzrderek
Copy link
Contributor

@xzrderek xzrderek commented Nov 15, 2025

Note

Adds Winston-based Fireworks logging with Vercel integration, introduces TS status/exception/types modules with re-exports, and updates build config to emit dist; also tweaks exception typing and a test filename/assertion.

  • TypeScript package:
    • Logging: Add logging/fireworks-transport.ts (Winston transport to Fireworks), logging/logger.ts (configured logger, createRolloutLogger), and logging/fireworks-vercel.ts (Vercel withFireworksLogging).
    • Models: Add models/status.ts, models/exceptions.ts (OpenAI error mapping, retry helper), and models/types.ts (zod schemas/types). Replace prior inline schemas by re-exporting from index.ts.
    • Build/Config: Update package.json to output dist (main/types/exports, files, deps, version), and tsconfig.json to emit declarations to dist.
  • Quickstart server (TS):
    • Refine exception map typing; call exception constructors without status code in exceptionForStatusCode.
  • Tests:
    • Adjust non-test-prefixed discovery test to use unique filename and assert accordingly.

Written by Cursor Bugbot for commit b9ff0d8. This will update automatically on new commits. Configure here.

@xzrderek xzrderek requested review from benjibc and dphuang2 November 15, 2025 09:25
@xzrderek xzrderek requested a review from dphuang2 November 17, 2025 08:58
@xzrderek xzrderek merged commit 467fffb into main Nov 17, 2025
7 checks passed
@xzrderek xzrderek deleted the derekx/typescript-package-logger branch November 17, 2025 20:12
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.

2 participants