Move scripts to their own directory#34
Merged
kevinbackhouse merged 1 commit intoGitHubSecurityLab:mainfrom Jan 29, 2026
Merged
Conversation
Contributor
There was a problem hiding this comment.
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.shfromsrc/toscripts/and updated it to install the package and execute commands from the repo root - Moved
run_audit.shfromsrc/toscripts/audit/and changed it from calling the wrapper script to directly usingpython -m seclab_taskflow_agent - Updated README.md documentation to reflect the new script locations and remove the requirement to
cd srcbefore 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.
3eeb68b to
53b7d93
Compare
Contributor
There was a problem hiding this comment.
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.
5263cf5 to
7c5b497
Compare
Contributor
Author
p-
approved these changes
Jan 29, 2026
kiandadban
pushed a commit
to kiandadban/seclab-taskflows
that referenced
this pull request
May 4, 2026
Move scripts to their own directory
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.
This PR does two things:
run_seclab_agent.shscript so that you run it from the main directory of the repo, rather than having to firstcdinto thesrcdirectory. 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 runspip install -q -e /appso that python's importlib knows where to find the files.run_seclab_agent.shandrun_audit.shinto a newscriptsdirectory.After this change, you can run the
run_audit.shscript in docker like this:Or, you can run it in codespaces (or on bare metal) like this: