Skip to content

v1.0.0#1

Merged
mast1ff merged 5 commits into
mainfrom
v1.0.0
Jun 18, 2025
Merged

v1.0.0#1
mast1ff merged 5 commits into
mainfrom
v1.0.0

Conversation

@mast1ff

@mast1ff mast1ff commented Jun 18, 2025

Copy link
Copy Markdown
Member

This pull request introduces several changes to improve project configuration, automate workflows, and provide comprehensive examples for using the TypeFetcher library. The most significant updates include adding new .editorconfig and biome.json files, replacing ESLint with Biome, setting up CI and release workflows, and adding detailed usage examples for TypeFetcher.

Configuration Updates:

  • Added .editorconfig to standardize file formatting across the project, including rules for indentation, line endings, and trailing whitespace.
  • Introduced biome.json to configure Biome for linting, formatting, and JavaScript-specific rules, replacing ESLint.
  • Removed .eslintrc.json and updated .vscode/settings.json to use Biome for code actions on save. [1] [2]

Workflow Automation:

  • Added .github/workflows/ci.yaml for continuous integration, including build, lint, format, and test coverage steps.
  • Added .github/workflows/release.yaml to automate npm package releases triggered by version tags.

Examples and Documentation:

  • Added detailed usage examples for TypeFetcher, including basic, type-safe, Node.js, and advanced use cases, in examples/basic-usage.ts, examples/nodejs-usage.ts, and examples/signal-and-raw-usage.ts. [1] [2] [3]

Other Changes:

  • Updated LICENSE to reflect the current year.
  • Removed unnecessary files from .npmignore to streamline the npm package.

@mast1ff mast1ff self-assigned this Jun 18, 2025
@mast1ff mast1ff requested a review from Copilot June 18, 2025 18:07

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This pull request updates the project configuration and tooling (switching from ESLint to Biome), adds CI/release workflows, and introduces comprehensive usage examples while upgrading and refactoring the underlying TypeFetcher client and schema integration.

  • New configuration files (.editorconfig, biome.json) standardize code style and linting.
  • The TypeFetcher client is refactored with enhanced schema validation, improved response handling, and expanded examples covering various use cases.
  • Legacy code (e.g., ESLint config and axios-based client) is removed, and test suites are updated accordingly.

Reviewed Changes

Copilot reviewed 27 out of 33 changed files in this pull request and generated no comments.

Show a summary per file
File Description
vitest.config.ts Added test configuration with coverage thresholds.
tsup.config.ts Configured bundling for ESM/CJS output with type definitions.
src/types.ts, src/schema.ts Updated types and schema helpers to support Standard Schema integration.
src/client.ts Refactored the main client with endpoint registration, request validation, and error handling.
Various example files Provided comprehensive usage examples (basic, type-safe, Node.js, Signal/Raw usage).
CI/Release workflows & config Introduced Biome, CI automation, and updated package metadata to v1.0.0.
Comments suppressed due to low confidence (2)

src/client.ts:226

  • [nitpick] Consider adding an inline comment here to document that the response is cloned to preserve the raw response data for later use, ensuring clarity on why cloning occurs before consuming the response body.
		const responseClone = response.clone();

src/client.ts:238

  • Consider providing a more descriptive error message instead of undefined to facilitate debugging when a non-ok HTTP response is encountered.
			throw new TypeFetcherError(response.status, response.statusText, undefined, errorData);

@mast1ff mast1ff merged commit d27deed into main Jun 18, 2025
2 checks passed
@mast1ff mast1ff deleted the v1.0.0 branch June 18, 2025 18:35
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