Skip to content

Conversation

@jbellerb
Copy link
Contributor

Previously, disabled pages (marked as draft) could still be accessed at /{slug}.md. This also updates llms.txt to dynamically pull from the docs collection. This should avoid any drift between the human-focused and llm-focused pages in the future.

@github-actions
Copy link
Contributor

github-actions bot commented Jan 28, 2026

CLI Documentation Generator

Metric Value
CLI Version 0.0.1
Commands Generated 24
Tests ✅ 0/0 passed
📚 Commands documented
  • sprite login
  • sprite logout
  • sprite org auth
  • sprite org list
  • sprite org logout
  • sprite org keyring disable
  • sprite org keyring enable
  • sprite auth setup
  • sprite create
  • sprite use
  • sprite list
  • sprite destroy
  • sprite exec
  • sprite console
  • sprite checkpoint create
  • sprite checkpoint list
  • sprite checkpoint info
  • sprite checkpoint delete
  • sprite restore
  • sprite proxy
  • sprite url
  • sprite url update
  • sprite api
  • sprite upgrade

@jbellerb jbellerb force-pushed the jae/push-yxsqlumwltqm branch 3 times, most recently from 5c10513 to 725668f Compare January 28, 2026 22:00
@jbellerb
Copy link
Contributor Author

I need to fix my local biome installation...

@github-actions
Copy link
Contributor

github-actions bot commented Jan 28, 2026

Preview Deployment

Name URL
Preview https://pr-128-superfly-sprites-docs.fly.dev

Commit: ab1a16d

@github-actions
Copy link
Contributor

github-actions bot commented Jan 28, 2026

Lighthouse Results

URL Performance Accessibility Best Practices SEO
/ 🟢 92 🟢 100 🟢 96 🟢 100

@github-actions
Copy link
Contributor

github-actions bot commented Jan 28, 2026

E2E Test Results

✅ Tests success

Ran against: https://pr-128-superfly-sprites-docs.fly.dev

@jbellerb jbellerb force-pushed the jae/push-yxsqlumwltqm branch from 11ac22f to 0a886da Compare January 28, 2026 22:49
@kcmartin
Copy link
Collaborator

A few comments:

  1. Debug code left in (llms-full.txt.ts:42):
    console.log(sidebarItem);

  2. Logic bug - the console.warn on line 56 fires for ALL object items, not just failures:
    } console.warn( // <-- This runs even after successful tryPushDoc `Warning: Failed to identify sidebar style of ${sidebarItem.label}:`, );
    Should probably be in an else block or have a continue before it.

  3. URL format change - Links changed from /quickstart/ to /quickstart.md. Intentional?

Copy link
Collaborator

@kcmartin kcmartin left a comment

Choose a reason for hiding this comment

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

See comment for requested changes & overlap with #126

Previously, disabled pages (marked as draft) could still be accessed at
`/{slug}.md`. This also updates llms.txt to dynamically pull from the
docs collection. This should avoid any drift between the human-focused
and llm-focused pages in the future.
@jbellerb jbellerb force-pushed the jae/push-yxsqlumwltqm branch from 710e74f to 61b7257 Compare January 29, 2026 01:13
@jbellerb
Copy link
Contributor Author

Thanks Claude for catching the stray debug messages. Changing to .md is intentional since those are for LLM consumption. It may be worth setting up an automatic redirect with nginx, too.

Copy link
Collaborator

@kcmartin kcmartin left a comment

Choose a reason for hiding this comment

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

LGTM
@jbellerb to file follow up on sprite-env

```

**Options:**
- `-c, --comment <string>` - Optional comment to describe this checkpoint
Copy link
Collaborator

Choose a reason for hiding this comment

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

I got this error when trying to use -c:

flag provided but not defined: -c``


**Options:**
- `-h, --help` - Show help for this command
- `-h, --history <string>` - Filter by history version Include auto-generated checkpoints
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is -h really used for two different options? (history and help?)
Maybe you can elaborate further?

@jbellerb jbellerb merged commit 2eb0d79 into main Jan 30, 2026
5 checks passed
@jbellerb jbellerb deleted the jae/push-yxsqlumwltqm branch January 30, 2026 17:51
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.

3 participants