Script for initializing a new taskflows repo#40
Script for initializing a new taskflows repo#40kevinbackhouse wants to merge 3 commits intoGitHubSecurityLab:mainfrom
Conversation
There was a problem hiding this comment.
Pull request overview
This PR adds a shell script to initialize a new taskflows repository based on the seclab-taskflows template. The script automates the creation of a project skeleton by using hatch new and then copying configuration files, directory structures, and workflows from the seclab-taskflows repository.
Changes:
- Adds
scripts/create_repo.shto automate creation of new taskflows repositories - Script creates standard directory structure (configs, mcp_servers, personalities, prompts, taskflows, toolboxes)
- Copies shared files like .devcontainer, CODE_OF_CONDUCT.md, .gitignore, and GitHub workflow files
Comments suppressed due to low confidence (2)
scripts/create_repo.sh:27
- There is an extra space before the closing parenthesis in the dirname command. This should be
$(dirname "${BASH_SOURCE[0]}")without the space to avoid potential issues with path resolution.
export SECLAB_TASKFLOWS="$(realpath $(dirname "${BASH_SOURCE[0]}" )/..)"
scripts/create_repo.sh:61
- The
uv addcommand uses the--activeand--frozenflags. The--frozenflag typically prevents updating the lockfile, but this is a new repository without an existing lockfile. This combination of flags may not work as expected. Consider using justuv add seclab-taskflowsor verify these flags are appropriate for a new project initialization.
uv add seclab-taskflows --active --frozen
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
722015b to
07f6093
Compare
07f6093 to
5258bf5
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 4 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>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.
Comments suppressed due to low confidence (1)
scripts/create_repo.sh:46
- The command substitution should be quoted to handle cases where the path contains spaces or special characters. The find command result should be quoted: SRCDIR="$(dirname "$(find . -name about.py)")"
SRCDIR="$(dirname $(find . -name __about__.py))"
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This script creates the skeleton for a new taskflows repo. It starts by calling
hatch newand then copies a few extra files from https://github.com/GitHubSecurityLab/seclab-taskflows.