Merged
Conversation
npx fastedge-debug now aligns with VsCode extension port management
There was a problem hiding this comment.
Pull request overview
This PR updates the FastEdge debugger server/CLI startup behavior (workspace anchoring, port selection, and logging), aligns Proxy-WASM property encoding with production behavior, and refreshes documentation/context files to match the new behavior.
Changes:
- Make the debugger server resolve config/port-file roots from
WORKSPACE_PATHorprocess.cwd(), add port auto-increment probing (5179–5188), and change startup logs to stderr. - Encode
response.status(and aliases) as a 2-byte big-endianu16inproxy_get_propertyto match Envoy/proxy-wasm. - Update integration tests and docs to reflect the new startup + property-encoding behavior and improve script portability.
Reviewed changes
Copilot reviewed 18 out of 20 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| server/server.ts | Switch config/port roots to `WORKSPACE_PATH |
| server/runner/HostFunctions.ts | Encode response.status property as 2-byte big-endian u16 |
| server/tests/integration/cdn-apps/property-access/response-properties.test.ts | Adjust assertions to no longer expect UTF-8 text for response.status |
| bin/fastedge-debug.js | Add app-root walk-up resolution and set WORKSPACE_PATH before loading the server |
| fastedge-plugin-source/generate-llms-txt.sh | Make package-name detection language-agnostic (Node/Rust/Python/fallback) |
| fastedge-plugin-source/check-copilot-sync.sh | Replace grep -P usage with POSIX awk and parse manifest via jq |
| docs/DEBUGGER.md | Document stderr startup logs, workspace discovery, port auto-increment, and port file location |
| docs/quickstart.md | Update config placement to .fastedge-debug/ and adjust $schema path |
| docs/API.md | Update /api/send typing notes and clarify response constraints |
| docs/RUNNER.md | Reformat tables and expand IStateManager signature documentation |
| docs/TEST_CONFIG.md | Table formatting + dotenv path resolution wording |
| docs/TEST_FRAMEWORK.md | Minor formatting tweaks in examples |
| docs/WEBSOCKET.md | Table alignment/formatting cleanup |
| context/VSCODE_BUNDLING.md | Document April 2026 auto-start + port probing architecture |
| context/features/CROSS_PLATFORM.md | Note WORKSPACE_PATH default + port probing changes |
| context/CONTEXT_INDEX.md | Update port behavior and add server-startup architecture index entry |
| context/CHANGELOG.md | Add changelog entries for CLI root resolution + server startup changes |
| context/FUTURE_ENHANCEMENTS.md | Add proposal for content-type aware response preview |
| CLAUDE.md | Add governance note and update port behavior references |
| AGENTS.md | Replace placeholder with structured agent policy header and rules |
Collaborator
Author
|
🎉 This PR is included in version 0.1.5 🎉 The release is available on: Your semantic-release bot 📦🚀 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.