Skip to content

Commit dc43795

Browse files
claudeAgentEnder
authored andcommitted
fix(docs-site): address PR review comments
- rehypeBaseUrl: skip protocol-relative URLs (//...) and hrefs that already equal the base path exactly - Fix grammar typo: "more abstract that" → "more abstract than" - Fix verification date: April 2025 → April 2026 - Fix GitHub owner casing: agentender → AgentEnder - Fix brand casing: "CLI forge" → "CLI Forge" https://claude.ai/code/session_01TQedBZpMs8uNKo28i8M65J
1 parent 6f84956 commit dc43795

File tree

4 files changed

+10
-5
lines changed

4 files changed

+10
-5
lines changed

docs-site/docs/guides/comparison.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ nav:
1111
1212
The Node.js ecosystem has many CLI building libraries, each with different design goals and tradeoffs. This guide compares CLI Forge with the most popular alternatives to help you choose the right tool for your project.
1313

14-
Comparisons were last verified against official documentation in April 2025. See the [version reference](#version-reference) at the bottom of this page for the specific versions compared.
14+
Comparisons were last verified against official documentation in April 2026. See the [version reference](#version-reference) at the bottom of this page for the specific versions compared.
1515

1616
## Feature matrix
1717

@@ -362,7 +362,7 @@ CLI Forge provides everything that `util.parseArgs` deliberately excludes:
362362

363363
## Links and references
364364

365-
- **CLI Forge**[Documentation](https://craigory.dev/cli-forge/) · [GitHub](https://github.com/agentender/cli-forge) · [npm](https://www.npmjs.com/package/cli-forge)
365+
- **CLI Forge**[Documentation](https://craigory.dev/cli-forge/) · [GitHub](https://github.com/AgentEnder/cli-forge) · [npm](https://www.npmjs.com/package/cli-forge)
366366
- **yargs**[Documentation](https://yargs.js.org/) · [GitHub](https://github.com/yargs/yargs) · [npm](https://www.npmjs.com/package/yargs)
367367
- **commander**[GitHub](https://github.com/tj/commander.js) · [npm](https://www.npmjs.com/package/commander)
368368
- **oclif**[Documentation](https://oclif.io/) · [GitHub](https://github.com/oclif/oclif) · [npm](https://www.npmjs.com/package/oclif)

docs-site/docs/guides/generate-docs.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ nav:
77

88
# Autogenerate CLI documentation
99

10-
Any CLI that is written with CLI forge can use `npx cli-forge generate-docs` to generate documentation.
10+
Any CLI that is written with CLI Forge can use `npx cli-forge generate-docs` to generate documentation.
1111

1212
By default, the command will generate documentation for your CLI in markdown. The templates for the markdown are not customizable.
1313

docs-site/server/utils/markdown.ts

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,12 @@ function rehypeBaseUrl(): (tree: Root) => void {
3030
visitElement(tree, (node) => {
3131
if (node.tagName === 'a' && typeof node.properties?.href === 'string') {
3232
const href = node.properties.href;
33-
if (href.startsWith('/') && !href.startsWith(BASE_URL + '/')) {
33+
if (
34+
href.startsWith('/') &&
35+
!href.startsWith('//') &&
36+
href !== BASE_URL &&
37+
!href.startsWith(BASE_URL + '/')
38+
) {
3439
node.properties.href = BASE_URL + href;
3540
}
3641
}

examples/object-dot-notation-simple.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
// > For a more detailed example showcasing defaults, required properties, validation, and JSON input,
1010
// > see the `object-arguments` multi-file example.
1111
//
12-
// > Note: This example is a bit more abstract that the others, as real world use cases for object-valued
12+
// > Note: This example is a bit more abstract than the others, as real world use cases for object-valued
1313
// > options and especially nested objects are less common. This example is included to demonstrate the
1414
// > flexibility of the CLI Forge APIs and the ability to handle complex option structures with type safety.
1515
// test:

0 commit comments

Comments
 (0)