Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 9, 2025

Summary

npm disallows package names too similar to existing packages. afetch conflicts with a-fetch, requiring the rename to what-the-fetch.

Details

Updated package name across all references:

  • Package identifiers: jsr.json (@hckhanh/what-the-fetch)
  • Documentation: README, CHANGELOG, all .mdx files in docs
  • Source code: JSDoc import examples in src/ files
  • Website metadata: SEO, Open Graph tags, navigation titles
  • GitHub templates: Repository URLs
  • Changeset: Updated .changeset/seven-meals-grin.md with comprehensive description including breaking changes and migration instructions

Example import change:

// Before
import { createFetch } from 'afetch';

// After
import { createFetch } from 'what-the-fetch';

The changeset file includes:

  • Explanation of the rename reason
  • Breaking changes documentation
  • Migration guide for npm and JSR users

All tests pass. No functional changes.

Original prompt

The package name afetch is too similar to a-fetch package, which is not allowed by npm. So we should rename our package from afetch to what-the-fetch accross all files. Thanks!


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI changed the title [WIP] Rename package afetch to what-the-fetch Rename package from afetch to what-the-fetch Nov 9, 2025
Copilot AI requested a review from hckhanh November 9, 2025 07:49
Copilot finished work on behalf of hckhanh November 9, 2025 07:49
@codecov
Copy link

codecov bot commented Nov 9, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (3ea27fc) to head (fe7b7d8).
⚠️ Report is 2 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff            @@
##              main        #9   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            3         3           
  Lines           40        40           
  Branches        13        13           
=========================================
  Hits            40        40           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@codspeed-hq
Copy link

codspeed-hq bot commented Nov 9, 2025

CodSpeed Performance Report

Merging #9 will not alter performance

Comparing copilot/rename-afetch-to-what-the-fetch (fe7b7d8) with main (6f915ca)1

Summary

✅ 2 untouched
⏩ 157 skipped2

Footnotes

  1. No successful run was found on main (468706e) during the generation of this report, so 6f915ca was used instead as the comparison base. There might be some changes unrelated to this pull request in this report.

  2. 157 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@hckhanh hckhanh marked this pull request as ready for review November 9, 2025 07:52
Copilot AI review requested due to automatic review settings November 9, 2025 07:52
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR renames the package from "afetch" to "what-the-fetch" to comply with npm naming requirements, as "afetch" is too similar to the existing "a-fetch" package.

  • Updated package identifiers in package.json, jsr.json, and bun.lock
  • Renamed all references in documentation files (README, CHANGELOG, docs content, and website metadata)
  • Updated import examples in JSDoc comments across source files

Reviewed Changes

Copilot reviewed 15 out of 16 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/utils.ts Updated JSDoc comment reference from "afetch" to "what-the-fetch"
src/types.ts Updated JSDoc comment reference from "afetch" to "what-the-fetch"
src/index.ts Updated import path examples from "@hckhanh/afetch" to "@hckhanh/what-the-fetch"
jsr.json Updated package name from "@hckhanh/afetch" to "@hckhanh/what-the-fetch"
docs/src/lib/layout.shared.tsx Updated GitHub URL and navigation title to "what-the-fetch"
docs/src/app/og/docs/[...slug]/route.tsx Updated site name in OpenGraph image generation
docs/src/app/layout.tsx Updated SEO metadata titles and keywords
docs/package.json Updated package name and description
docs/content/docs/index.mdx Renamed all references throughout introduction documentation
docs/content/docs/getting-started.mdx Updated installation commands and all references
docs/content/docs/api-reference.mdx Updated all code examples and descriptive text
bun.lock Updated workspace name reference
README.md Updated title, badges, and all references to the new package name
CHANGELOG.md Updated changelog title and release notes
.github/PULL_REQUEST_TEMPLATE.md Updated repository URL

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@hckhanh
Copy link
Owner

hckhanh commented Nov 9, 2025

@copilot update .changeset/seven-meals-grin.md

Copy link
Contributor Author

Copilot AI commented Nov 9, 2025

@copilot update .changeset/seven-meals-grin.md

Updated the changeset file with a comprehensive description of the package rename, including breaking changes and migration instructions. (b34ccc4)

Copilot AI requested a review from hckhanh November 9, 2025 08:01
Copilot finished work on behalf of hckhanh November 9, 2025 08:01
@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 9, 2025

@hckhanh hckhanh merged commit 6f915ca into main Nov 9, 2025
12 checks passed
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