Skip to content

Script for initializing a new taskflows repo#40

Open
kevinbackhouse wants to merge 3 commits intoGitHubSecurityLab:mainfrom
kevinbackhouse:create-repo
Open

Script for initializing a new taskflows repo#40
kevinbackhouse wants to merge 3 commits intoGitHubSecurityLab:mainfrom
kevinbackhouse:create-repo

Conversation

@kevinbackhouse
Copy link
Contributor

This script creates the skeleton for a new taskflows repo. It starts by calling hatch new and then copies a few extra files from https://github.com/GitHubSecurityLab/seclab-taskflows.

Copilot AI review requested due to automatic review settings February 5, 2026 21:50
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.sh to 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 add command uses the --active and --frozen flags. The --frozen flag 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 just uv add seclab-taskflows or 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.

Copilot AI review requested due to automatic review settings February 6, 2026 11:50
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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>
Copilot AI review requested due to automatic review settings February 6, 2026 11:54
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

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