Skip to content

Conversation

Copy link

Copilot AI commented Nov 2, 2025

Replaced sample fumadocs content with production documentation for the fast-url library per web/INSTRUCTION.md.

Documentation Structure

  • Introduction - Fork rationale, performance improvements (5-20% faster than urlcat), key features, use cases
  • Getting Started - Installation across package managers, usage patterns, step-by-step API client tutorial, Unicode/special character handling
  • API Reference - Complete function signatures with examples (createUrl, query, subst, join), advanced patterns (pagination, dynamic routing)

Technical Changes

  • SEO optimization - Added Open Graph, Twitter cards, and structured metadata
  • Branding - Updated navigation with fast-url title and GitHub link
  • Home page - Redirects directly to /docs for immediate access to documentation
  • Documentation naming - All examples use createUrl instead of urlcat for consistency
  • Removed - Sample documentation files (test.mdx, v1/, v2/)
  • Build fix - Removed Google Fonts dependency (network restricted in CI environments)
  • CodeCov configuration - Added codecov.yml to exclude web/ directory from coverage reports

Components Utilized

Fumadocs UI: Cards, Tabs, Steps, Callouts, syntax highlighting with copy-to-clipboard

Preview

Documentation

All documentation pages use createUrl consistently throughout examples and API references.

Original prompt

read the web/INSTRUCTION.md and follow its instruction


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI changed the title [WIP] Follow instructions in INSTRUCTION.md file Create comprehensive documentation for fast-url using fumadocs Nov 2, 2025
Copilot AI requested a review from hckhanh November 2, 2025 00:51
Copilot finished work on behalf of hckhanh November 2, 2025 00:51
@hckhanh
Copy link
Owner

hckhanh commented Nov 2, 2025

@copilot I think instead of using import urlcat, we should change to import createUrl

Copy link
Author

Copilot AI commented Nov 2, 2025

@copilot I think instead of using import urlcat, we should change to import createUrl

Changed in commit 1fbe2cc. All documentation now uses import createUrl instead of import urlcat, and all function calls have been updated from urlcat() to createUrl() throughout the Introduction, Getting Started, and API Reference pages.

Additionally, the home page now redirects directly to /docs as requested.

Documentation with createUrl

@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 2, 2025

Copilot finished work on behalf of hckhanh November 2, 2025 02:57
hckhanh and others added 3 commits November 2, 2025 10:07
- Simplify `baseOptions` by removing redundant `links` array and directly integrating `githubUrl`.
- Overhaul documentation content in `/docs`:
  - Replace `introduction.mdx` with comprehensive updates in `index.mdx`.
  - Introduce new structure, emphasizing `fast-url` features, performance, and use-cases.
  - Add `Tabs` and `Cards` for better content accessibility and readability.
- Improve alignment with modern JavaScript/TypeScript practices.
- Add a permanent redirect from `/` to `/docs` in `next.config.mjs`.
- Remove unused `(home)` layout and page components.
- Update Open Graph metadata for better brand alignment (`site` updated to "fast-url").
@codecov
Copy link

codecov bot commented Nov 2, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (docs@1a6de2d). Learn more about missing BASE report.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff            @@
##             docs       #49   +/-   ##
========================================
  Coverage        ?   100.00%           
========================================
  Files           ?         3           
  Lines           ?       191           
  Branches        ?        63           
========================================
  Hits            ?       191           
  Misses          ?         0           
  Partials        ?         0           

☔ 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 2, 2025

CodSpeed Performance Report

Merging #49 will not alter performance

Comparing copilot/follow-instruction-in-web-directory (dcc5715) with main (8d07e2d)1

Summary

✅ 157 untouched
⏩ 4 skipped2

Footnotes

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

  2. 4 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 and others added 3 commits November 2, 2025 10:18
…g files

- Standardize indentation to 2 spaces across the entire codebase (JSON, TS, TSX, and config files).
- Replace double quotes with single quotes in JavaScript/TypeScript files for alignment with modern conventions.
- Improve readability in metadata and configuration files.
- No functional changes.
@hckhanh
Copy link
Owner

hckhanh commented Nov 2, 2025

@copilot update codecov to ignore web/

Copy link
Author

Copilot AI commented Nov 2, 2025

@copilot update codecov to ignore web/

Added in commit dcc5715. Created codecov.yml configuration file that ignores the web/ directory from coverage reports.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 2, 2025

@hckhanh hckhanh marked this pull request as ready for review November 2, 2025 03:28
Copilot AI review requested due to automatic review settings November 2, 2025 03:28
Copilot finished work on behalf of hckhanh November 2, 2025 03:28
Copy link

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 adds comprehensive documentation for the fast-url library and updates branding from "My App" to "fast-url" throughout the web application. The changes include extensive MDX documentation files, configuration updates for code formatting (tabs to spaces), and the removal of placeholder content.

Key Changes:

  • Added complete documentation (introduction, getting started, API reference)
  • Updated branding and metadata throughout the web application
  • Standardized code formatting from tabs to spaces
  • Removed placeholder/demo content and routes

Reviewed Changes

Copilot reviewed 27 out of 29 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
web/biome.json Updated biome configuration extends property
web/tsconfig.json Reformatted with spaces instead of tabs
web/package.json Reformatted with spaces instead of tabs
web/postcss.config.mjs Reformatted with spaces instead of tabs
web/source.config.ts Reformatted with spaces instead of tabs
web/next.config.mjs Added homepage redirect and reformatted
web/src/**/*.tsx Updated imports to single quotes and spaces, removed Inter font
web/src/lib/layout.shared.tsx Updated branding from "My App" to "fast-url"
web/src/app/layout.tsx Added comprehensive metadata and removed custom font
web/src/app/og/docs/[...slug]/route.tsx Updated site name from "My App" to "fast-url"
web/src/app/(home)/*.tsx Removed home layout and page components
web/content/docs/*.mdx Added comprehensive documentation and removed placeholders
web/content/docs/meta.json Updated documentation structure
package.json Updated homepage URL to documentation site
codecov.yml Added configuration to exclude web directory
.changeset/slimy-frogs-roll.md Added changeset entry

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

}
}
"$schema": "../node_modules/@biomejs/biome/configuration_schema.json",
"extends": "//",
Copy link

Copilot AI Nov 2, 2025

Choose a reason for hiding this comment

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

The extends value '//' appears to be incorrect. This should likely reference a parent biome configuration file path like '../biome.json' (as shown in the original code) or be removed entirely if not extending a parent config.

Suggested change
"extends": "//",

Copilot uses AI. Check for mistakes.
@hckhanh hckhanh merged commit 9e44821 into docs Nov 2, 2025
16 checks passed
@hckhanh hckhanh deleted the copilot/follow-instruction-in-web-directory branch November 2, 2025 03:31
@hckhanh hckhanh restored the copilot/follow-instruction-in-web-directory branch November 2, 2025 03:32
@hckhanh hckhanh deleted the copilot/follow-instruction-in-web-directory branch November 2, 2025 03:34
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