Skip to content

Minimal local run scripts#35

Merged
em3s merged 1 commit intomainfrom
init/run-local
Jan 13, 2026
Merged

Minimal local run scripts#35
em3s merged 1 commit intomainfrom
init/run-local

Conversation

@em3s
Copy link
Copy Markdown
Contributor

@em3s em3s commented Jan 13, 2026 •
edited
Loading

Add minimal local run scripts for Actionbase.

  • Ignore .run/ in .gitignore
  • Provide run_local.sh and stop_local.sh
  • Prevent duplicate runs using a PID file
  • Wait for port 8080 before reporting startup success
  • Store logs and PID under .run/
  • Keep output limited to success/failure, log path, and PID path

There is room for further improvement, but this keeps it minimal for now.

@em3s
feat: minimal local run scripts
96e4efe 
Add minimal local run scripts for Actionbase.

- Provide `run_local.sh` and `stop_local.sh`
- Prevent duplicate runs using a PID file
- Wait for port 8080 before reporting startup success
- Store logs and PID under `.run/`
- Keep output limited to success/failure, log path, and PID path

There is room for further improvement, but this keeps it minimal for now.
@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. enhancement New feature or request labels Jan 13, 2026
@em3s
Copy link
Copy Markdown
Contributor Author

em3s commented Jan 13, 2026

You can verify as follows:

$ ./bin/run_local.sh 
Connection to localhost port 8080 [tcp/http-alt] succeeded!
✅ Actionbase server started on 8080 🚀
log: .run/logs/actionbase.log
pid: .run/actionbase.pid

$ curl localhost:8080
{"status":"UP","message":"Actionbase is running","version":"0.0.1-SNAPSHOT","timestamp":"2026-01-13T08:41:16.085157Z"}

$ ./bin/stop_local.sh 
✅ Actionbase server st]opped
pid: .run/actionbase.pid

@em3s em3s mentioned this pull request Jan 13, 2026
Closed
10 tasks
zipdoki
zipdoki approved these changes Jan 13, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@zipdoki zipdoki left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jan 13, 2026
@em3s em3s mentioned this pull request Jan 13, 2026
Open
@em3s em3s merged commit eb7f2dd into main Jan 13, 2026
3 checks passed
@em3s em3s mentioned this pull request Jan 13, 2026
Closed
@em3s em3s self-assigned this Jan 19, 2026
@em3s em3s changed the title feat: minimal local run scripts Minimal local run scripts Feb 12, 2026
@em3s em3s deleted the init/run-local branch March 1, 2026 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@zipdoki zipdoki zipdoki approved these changes

Assignees

@em3s em3s

Labels

enhancement New feature or request lgtm This PR has been approved by a maintainer size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@em3s @zipdoki