Skip to content

Rework new feature command to store request source in feature.json - #11

Merged
darthsharp merged 3 commits into
mainfrom
feature/rework-new-feature
Oct 26, 2026
Merged

darthsharp merged 3 commits into
mainfrom
feature/rework-new-feature

Conversation

@darthsharp

Copy link
Copy Markdown
Contributor

No description provided.

…plement external process handling and GitHub CLI integration**

- Removed `FeatureRequestTemplate` and `request.template.md` to streamline feature creation logic.
- Added `ExternalProcessRunner` and `ExternalProcessResult` for managing external processes with captured output.
- Introduced `GhCommandRunner`, `GhCommandResult`, and `GhExecutableLocator` to integrate GitHub CLI (`gh`) functionality.
- Added extensive tests for GH command execution, process handling, and executable discovery.
…lated tests for simplification**

- Removed unused `HttpUrlContentReader`, `GhCommandResult`, and `GitCommandResult` classes to streamline the codebase.
- Deleted associated tests for `HttpUrlContentReader`, `GhNotFoundException`, and related functionality.
…tests**

- Updated `HttpUrlContentReader` to delegate redirect handling entirely to `HttpClient`.
- Allowed `us-ascii` as a valid charset (interpreted as UTF-8).
- Refactored tests to remove redundant redirect scenarios and focus on edge cases.
- Improved error handling to provide clearer reasons for redirect failures.
- Reworked `FeatureIdFactory` to enforce size limits for Markdown issues and enhanced control character validation.
- Removed unused helpers and outdated logic for redirect handling.
@darthsharp
darthsharp merged commit c56580a into main Oct 26, 2026
7 checks passed
@darthsharp
darthsharp deleted the feature/rework-new-feature branch October 26, 2026 13:16
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