Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR removes the legacy “package install + local cache” workflow-addition path and shifts workflow resolution to fetch remote workflows directly from GitHub, while adding/expanding support for local workflow specs and local wildcard expansion.
Changes:
- Remove repository “package” install/list/discovery code paths and related tests.
- Resolve remote workflows by fetching content from GitHub (no cloning) and pass pre-fetched content through the add pipeline.
- Add local-path detection helper and local wildcard expansion logic; update tests accordingly.
Reviewed changes
Copilot reviewed 23 out of 24 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| pkg/workflow/compiler_activation_jobs.go | Pure formatting/gofmt adjustments in activation sparse checkout helper. |
| pkg/parser/remote_fetch.go | Adds exported wrappers for internal GitHub fetch/ref-resolution helpers. |
| pkg/cli/trial_repository.go | Switches trial-mode remote workflow install from “install package + add” to direct fetch + write. |
| pkg/cli/spec.go | Adds isLocalWorkflowPath and expands local-path detection in parsing/stringification. |
| pkg/cli/packages.go | Removes package install/list/discovery functionality; refactors include dependency collection naming/struct fields. |
| pkg/cli/packages_test.go | Updates tests to use renamed local include dependency collector. |
| pkg/cli/packages_fallback_test.go | Deletes tests for removed fallback path/package workflow resolution behavior. |
| pkg/cli/local_workflow_trial_test.go | Updates local path assertions to use isLocalWorkflowPath. |
| pkg/cli/local_workflow_integration_test.go | Updates expectations to new WorkflowSourceInfo shape (IsLocal). |
| pkg/cli/commands_utils_test.go | Removes tests tied to deleted copyMarkdownFiles helper and related import. |
| pkg/cli/commands_test.go | Removes tests for deleted InstallPackage behavior. |
| pkg/cli/add_workflow_resolution.go | Removes “install packages then read from disk” resolution; fetches remote content directly; adds local-only wildcard expansion. |
| pkg/cli/add_workflow_repository.go | Deletes repo-only “list workflows” interactive/table UX path. |
| pkg/cli/add_workflow_pr.go | PR-creation path now operates on pre-resolved workflows. |
| pkg/cli/add_workflow_not_found_test.go | Deletes tests for removed “display available workflows” behavior. |
| pkg/cli/add_wildcard_test.go | Updates wildcard tests for local wildcard expansion and removes package discovery tests. |
| pkg/cli/add_repo_only_test.go | Removes tests that depended on deleted package workflow listing / repo-only handling. |
| pkg/cli/add_interactive_workflow_test.go | Tweaks expectations around getWorkflowStatuses return values in test environment. |
| pkg/cli/add_interactive_orchestrator.go | Removes outdated comment referencing removed add path. |
| pkg/cli/add_gitattributes_test.go | Updates tests to call new addWorkflows([]*ResolvedWorkflow, ...) path. |
| pkg/cli/add_command_test.go | Updates expectation: repo-only specs now error with format guidance. |
| pkg/cli/add_command_table_test.go | Deletes table-display tests for removed workflow listing UX. |
| pkg/cli/add_command.go | Removes repo-only listing + package install flow; “add” now resolves to pre-fetched workflows and adds them; documents local specs and local wildcards. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 27 out of 27 changed files in this pull request and generated 6 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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.
Summary
add_command.gofileChanges
Motivation
The changes streamline the workflow addition process, removing outdated functionality and making the code more focused and maintainable.