fix(freebsd): move chmod after daemon command in dashboard rc.d #9

Closed

clawdie wants to merge 0 commits from pr-7 into main

pull from: pr-7

merge into: clawdie:main

clawdie:main

Conversation 0 Commits - Files changed -

clawdie commented 2026-06-21 12:25:45 +02:00

Owner

Copy link

The chmod 644 on pidfiles was injected mid-command between daemon(8)
and its arguments (--host, --port, etc.), causing the shell to pass
'chmod' as a subcommand argument to 'hermes dashboard', which
instantly crashed. Move it after the complete daemon invocation.

This caused the 502 Bad Gateway on ai.clawdie.si after restart —
dashboard process never started because it rejected 'chmod' as an
invalid dashboard subcommand.

What does this PR do?

Related Issue

Fixes #

Type of Change

• 🐛 Bug fix (non-breaking change that fixes an issue)
• ✨ New feature (non-breaking change that adds functionality)
• 🔒 Security fix
• 📝 Documentation update
• ✅ Tests (adding or improving test coverage)
• ♻️ Refactor (no behavior change)
• 🎯 New skill (bundled or hub)

Changes Made

How to Test

Checklist

Code

• I've read the Contributing Guide
• My commit messages follow Conventional Commits (fix(scope):, feat(scope):, etc.)
• I searched for existing PRs to make sure this isn't a duplicate
• My PR contains only changes related to this fix/feature (no unrelated commits)
• I've run pytest tests/ -q and all tests pass
• I've added tests for my changes (required for bug fixes, strongly encouraged for features)
• I've tested on my platform:

Documentation & Housekeeping

• I've updated relevant documentation (README, docs/, docstrings) — or N/A
• I've updated cli-config.yaml.example if I added/changed config keys — or N/A
• I've updated CONTRIBUTING.md or AGENTS.md if I changed architecture or workflows — or N/A
• I've considered cross-platform impact (Windows, macOS) per the compatibility guide — or N/A
• I've updated tool descriptions/schemas if I changed tool behavior — or N/A

For New Skills

• This skill is broadly useful to most users (if bundled) — see Contributing Guide
• SKILL.md follows the standard format (frontmatter, trigger conditions, steps, pitfalls)
• No external dependencies that aren't already available (prefer stdlib, curl, existing Hermes tools)
• I've tested the skill end-to-end: hermes --toolsets skills -q "Use the X skill to do Y"

Screenshots / Logs

The chmod 644 on pidfiles was injected mid-command between daemon(8) and its arguments (--host, --port, etc.), causing the shell to pass 'chmod' as a subcommand argument to 'hermes dashboard', which instantly crashed. Move it after the complete daemon invocation. This caused the 502 Bad Gateway on ai.clawdie.si after restart — dashboard process never started because it rejected 'chmod' as an invalid dashboard subcommand. ## What does this PR do? <!-- Describe the change clearly. What problem does it solve? Why is this approach the right one? --> ## Related Issue <!-- Link the issue this PR addresses. If no issue exists, consider creating one first. --> Fixes # ## Type of Change <!-- Check the one that applies. --> - [ ] 🐛 Bug fix (non-breaking change that fixes an issue) - [ ] ✨ New feature (non-breaking change that adds functionality) - [ ] 🔒 Security fix - [ ] 📝 Documentation update - [ ] ✅ Tests (adding or improving test coverage) - [ ] ♻️ Refactor (no behavior change) - [ ] 🎯 New skill (bundled or hub) ## Changes Made <!-- List the specific changes. Include file paths for code changes. --> - ## How to Test <!-- Steps to verify this change works. For bugs: reproduction steps + proof that the fix works. --> 1. 2. 3. ## Checklist <!-- Complete these before requesting review. --> ### Code - [ ] I've read the [Contributing Guide](https://github.com/NousResearch/hermes-agent/blob/main/CONTRIBUTING.md) - [ ] My commit messages follow [Conventional Commits](https://www.conventionalcommits.org/) (`fix(scope):`, `feat(scope):`, etc.) - [ ] I searched for [existing PRs](https://github.com/NousResearch/hermes-agent/pulls) to make sure this isn't a duplicate - [ ] My PR contains **only** changes related to this fix/feature (no unrelated commits) - [ ] I've run `pytest tests/ -q` and all tests pass - [ ] I've added tests for my changes (required for bug fixes, strongly encouraged for features) - [ ] I've tested on my platform: <!-- e.g. Ubuntu 24.04, macOS 15.2, Windows 11 --> ### Documentation & Housekeeping <!-- Check all that apply. It's OK to check "N/A" if a category doesn't apply to your change. --> - [ ] I've updated relevant documentation (README, `docs/`, docstrings) — or N/A - [ ] I've updated `cli-config.yaml.example` if I added/changed config keys — or N/A - [ ] I've updated `CONTRIBUTING.md` or `AGENTS.md` if I changed architecture or workflows — or N/A - [ ] I've considered cross-platform impact (Windows, macOS) per the [compatibility guide](https://github.com/NousResearch/hermes-agent/blob/main/CONTRIBUTING.md#cross-platform-compatibility) — or N/A - [ ] I've updated tool descriptions/schemas if I changed tool behavior — or N/A ## For New Skills <!-- Only fill this out if you're adding a skill. Delete this section otherwise. --> - [ ] This skill is **broadly useful** to most users (if bundled) — see [Contributing Guide](https://github.com/NousResearch/hermes-agent/blob/main/CONTRIBUTING.md#should-the-skill-be-bundled) - [ ] SKILL.md follows the [standard format](https://github.com/NousResearch/hermes-agent/blob/main/CONTRIBUTING.md#skillmd-format) (frontmatter, trigger conditions, steps, pitfalls) - [ ] No external dependencies that aren't already available (prefer stdlib, curl, existing Hermes tools) - [ ] I've tested the skill end-to-end: `hermes --toolsets skills -q "Use the X skill to do Y"` ## Screenshots / Logs <!-- If applicable, add screenshots or log output showing the fix/feature in action. -->

clawdie added 1 commit 2026-06-21 12:25:47 +02:00

fix(freebsd): move chmod after daemon command in dashboard rc.d ... 9d54686c93

The chmod 644 on pidfiles was injected mid-command between daemon(8)
and its arguments (--host, --port, etc.), causing the shell to pass
'chmod' as a subcommand argument to 'hermes dashboard', which
instantly crashed. Move it after the complete daemon invocation.

This caused the 502 Bad Gateway on ai.clawdie.si after restart —
dashboard process never started because it rejected 'chmod' as an
invalid dashboard subcommand.

codex-osa added 1 commit 2026-06-21 12:29:48 +02:00

fix(docker): bump Node 22 → 24 LTS in Dockerfile ... 4d5a5746a3

Closes #4. Container image now matches the toolchain baseline
(Node 24 LTS across the matrix). Digest updated to current
node:24-bookworm-slim amd64.

codex-osa closed this pull request 2026-06-21 12:30:39 +02:00

Some checks failed

Hide all checks

History Check / check-common-ancestor (pull_request) Has been cancelled

Details

Lint (ruff + ty) / ruff + ty diff (pull_request) Has been cancelled

Details

Lint (ruff + ty) / Windows footguns (blocking) (pull_request) Has been cancelled

Details

Tests / test (1) (pull_request) Has been cancelled

Details

Typecheck / typecheck (ui-tui) (pull_request) Has been cancelled

Details

Supply Chain Audit / changes (pull_request) Has been cancelled

Details

Contributor Attribution Check / check-attribution (pull_request) Has been cancelled

Details

Docker Build and Publish / build-amd64 (pull_request) Has been cancelled

Details

Typecheck / typecheck (apps/shared) (pull_request) Has been cancelled

Details

Docker / shell lint / Lint Dockerfile (hadolint) (pull_request) Has been cancelled

Details

Docker / shell lint / Lint docker/ shell scripts (shellcheck) (pull_request) Has been cancelled

Details

Nix / nix (macos-latest) (pull_request) Has been cancelled

Details

Nix / nix (ubuntu-latest) (pull_request) Has been cancelled

Details

Docker Build and Publish / build-arm64 (pull_request) Has been cancelled

Details

Lint (ruff + ty) / ruff enforcement (blocking) (pull_request) Has been cancelled

Details

Typecheck / typecheck (apps/bootstrap-installer) (pull_request) Has been cancelled

Details

Tests / e2e (pull_request) Has been cancelled

Details

Typecheck / typecheck (apps/desktop) (pull_request) Has been cancelled

Details

Tests / test (2) (pull_request) Has been cancelled

Details

Tests / test (3) (pull_request) Has been cancelled

Details

Tests / test (4) (pull_request) Has been cancelled

Details

Tests / test (5) (pull_request) Has been cancelled

Details

Typecheck / typecheck (web) (pull_request) Has been cancelled

Details

Tests / test (6) (pull_request) Has been cancelled

Details

Docker Build and Publish / merge (pull_request) Has been cancelled

Details

Supply Chain Audit / Scan PR for critical supply chain risks (pull_request) Has been cancelled

Details

Supply Chain Audit / Check PyPI dependency upper bounds (pull_request) Has been cancelled

Details

Tests / save-durations (pull_request) Has been cancelled

Details

Pull request closed

This pull request cannot be reopened because the branch was deleted.

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels

No items

No labels

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

2 participants

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: clawdie/hermes-bsd#9

No description provided.