Merged
Conversation
Step instructions can now reference additional .md files placed in the steps/ directory. When slash commands are generated, references to these files (in backticks, markdown links, or quoted strings) are automatically transformed to relative paths from the project root. Changes: - Add _find_supplementary_files() to detect extra .md files in steps dir - Add _transform_md_references() to convert filename refs to full paths - Update Claude and Gemini templates to show supplementary files section - Document the feature in implement.md for deepwork_jobs - Add comprehensive tests for the new functionality
Contributor
|
I have read the CLA Document and I hereby sign the CLA 2 out of 3 committers have signed the DCO. |
Remove automatic path transformation code in favor of documenting that users should reference supplementary files using full paths from the project root (e.g., .deepwork/jobs/job_name/steps/reference.md). This is simpler and more explicit - no magic transformations needed. Changes: - Remove _find_supplementary_files() and _transform_md_references() from generator - Remove supplementary_files template section from both Claude and Gemini templates - Remove supplementary files tests - Add supplemental_file_references.md documentation file - Update implement.md to reference the new doc file with full path
The .deepwork/ directory should only be modified by `deepwork install`, not by manual file copies. Remove the supplementary files that were incorrectly added to the installed jobs directory. The supplementary file documentation remains in the source at src/deepwork/standard_jobs/deepwork_jobs/steps/ and will be installed when users run `deepwork install`.
Contributor
Author
|
I have read the CLA Document and I hereby sign the CLA |
Contributor
Author
|
I have read the CLA Document and I hereby sign the CLA |
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
2 participants
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.
Step instructions can now reference additional .md files placed in the steps/ directory. When slash commands are generated, references to these files (in backticks, markdown links, or quoted strings) are automatically transformed to relative paths from the project root.
Changes: