Skip to content

refactor: streamline file handling in HttpCommand#14

Merged
AxiosLeo merged 11 commits intomainfrom
refactor/http-command
May 2, 2026
Merged

refactor: streamline file handling in HttpCommand#14
AxiosLeo merged 11 commits intomainfrom
refactor/http-command

Conversation

@AxiosLeo
Copy link
Copy Markdown
Owner

@AxiosLeo AxiosLeo commented May 2, 2026

Note

Medium Risk
Moderate risk because it changes runtime behavior of the http CLI (directory reads, response headers, streaming lifecycle, and disables Koa static), which could affect file serving compatibility and edge cases.

Overview
Adds a new skills CLI command to install bundled AI “Skill” docs into Cursor/Claude skill directories, including logic to source skills from a local node_modules/@axiosleo/koapp/assets/skills when available (or fall back to the npx runner) and optional overwrite prompts/--force.

Bundles a large set of framework skills under assets/skills/* and documents usage in README.md.

Refactors HttpCommand file handling to use fs.promises.readdir with Dirents + stat for accurate directory detection (incl. symlinks), improves file responses (MIME handling via ctx.type, attachment fallback, Content-Length), adds stream error/close cleanup, prints a startup message, and disables the built-in Koa static server (static: false).

Reviewed by Cursor Bugbot for commit c8d87a5. Bugbot is set up for automated code reviews on this repo. Configure here.

…s.readdir and improving error handling for streams
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 77e197d. Configure here.

Comment thread commands/http.js Outdated
Comment thread commands/http.js Outdated
AxiosLeo added 10 commits May 2, 2026 21:05
…p, including installation instructions and core exports
…cation, and WebSocketApplication, including usage examples, configuration options, and connection management details
…including minimal setup, shared state management, CRUD operations, logging, and testing strategies
…mples, validation rules, and model serialization methods
…sage examples for response helpers, error handling, and custom headers
…ring route definitions, path parameters, validators, nested routers, and middleware usage
…ng usage patterns for Server-Sent Events, including progress updates, live logs, and authentication
…Claude, including source resolution, skill copying, and user prompts
…stallation and usage, including project and user scope options, bundled skills overview, and uninstall instructions
… symlink resolution and improve content type setting for Koa responses
@AxiosLeo AxiosLeo merged commit 59dd024 into main May 2, 2026
11 checks passed
@AxiosLeo AxiosLeo deleted the refactor/http-command branch May 2, 2026 13:21
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.

1 participant