Skip to content

Move scripts to their own directory#34

Merged
kevinbackhouse merged 1 commit intoGitHubSecurityLab:mainfrom
kevinbackhouse:scripts
Jan 29, 2026
Merged

Move scripts to their own directory#34
kevinbackhouse merged 1 commit intoGitHubSecurityLab:mainfrom
kevinbackhouse:scripts

Conversation

@kevinbackhouse
Copy link
Copy Markdown
Contributor

@kevinbackhouse kevinbackhouse commented Jan 28, 2026

This PR does two things:

  1. Modify the run_seclab_agent.sh script so that you run it from the main directory of the repo, rather than having to first cd into the src directory. This is currently a weird discrepancy between the docker experience and the codespaces experience which I want to fix. The main change is that the script now runs pip install -q -e /app so that python's importlib knows where to find the files.
  2. Move run_seclab_agent.sh and run_audit.sh into a new scripts directory.

After this change, you can run the run_audit.sh script in docker like this:

./scripts/run_seclab_agent.sh ./scripts/audit/run_audit.sh github/cmark-gfm

Or, you can run it in codespaces (or on bare metal) like this:

./scripts/audit/run_audit.sh github/cmark-gfm

Copilot AI review requested due to automatic review settings January 28, 2026 15:09
Copy link
Copy Markdown
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 reorganizes the repository structure by moving scripts from the src/ directory into a dedicated scripts/ directory and updates the main wrapper script to run from the repository root instead of requiring users to change into the src/ directory first.

Changes:

  • Moved run_seclab_agent.sh from src/ to scripts/ and updated it to install the package and execute commands from the repo root
  • Moved run_audit.sh from src/ to scripts/audit/ and changed it from calling the wrapper script to directly using python -m seclab_taskflow_agent
  • Updated README.md documentation to reflect the new script locations and remove the requirement to cd src before running commands

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
src/run_seclab_agent.sh Removed the old docker wrapper script from the src directory
src/run_audit.sh Removed the old audit script from the src directory
scripts/run_seclab_agent.sh Added new docker wrapper script that runs from repo root and installs package before execution
scripts/audit/run_audit.sh Added new audit script that directly invokes python module instead of calling wrapper
README.md Updated documentation to reference new script paths and removed instruction to cd into src directory

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread scripts/run_seclab_agent.sh Outdated
Comment thread README.md Outdated
Comment thread scripts/audit/run_audit.sh Outdated
Copy link
Copy Markdown
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 5 out of 5 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@kevinbackhouse kevinbackhouse marked this pull request as ready for review January 28, 2026 15:25
@kevinbackhouse
Copy link
Copy Markdown
Contributor Author

@p- @m-y-mo : Are you happy with this? It changes the docker workflow, but I think it's an improvement because now you can run the script from the main directory, rather than needing to cd into src.

@kevinbackhouse kevinbackhouse merged commit efbde89 into GitHubSecurityLab:main Jan 29, 2026
6 checks passed
@kevinbackhouse kevinbackhouse deleted the scripts branch January 30, 2026 18:35
kiandadban pushed a commit to kiandadban/seclab-taskflows that referenced this pull request May 4, 2026
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.

3 participants