docs(runbook): repo-paths aanmaken zonder install /dev/stdin #266

Merged

janpeter merged 3 commits from docs/repo-paths-command into main 2026-09-28 19:03:59 +02:00

Conversation 2 Commits 3 Files changed 2 +15 -2

janpeter commented 2026-09-28 17:25:21 +02:00

Owner

Copy link

macOS install(1) cannot read a pipe (/dev/stdin: Inappropriate file type or format). The runbook command now writes via sh -c with umask 027, then sets root:_jpops 0640. Docs only.

🤖 Generated with Claude Code

macOS `install(1)` cannot read a pipe (`/dev/stdin: Inappropriate file type or format`). The runbook command now writes via `sh -c` with `umask 027`, then sets `root:_jpops 0640`. Docs only. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

janpeter added 1 commit 2026-09-28 17:25:21 +02:00

docs(runbook): repo-paths aanmaken zonder install /dev/stdin (werkt niet op macOS) ... 7bbe083480

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

janpeter added 1 commit 2026-09-28 17:28:56 +02:00

docs: handmatige launcher-installatie en Git-beperkingen op mac-prod ... 1ebb373589

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

s4m-codex-reviewer approved these changes 2026-09-28 17:31:02 +02:00

Dismissed

s4m-codex-reviewer left a comment

Copy link

Verdict: APPROVED

Geen blokkerende findings.

• Info — deploy/mac-production/README.md:66: het runbook gebruikt nu een macOS-compatibele schrijfroute voor repo-paths en herstelt expliciet eigenaar en permissies.
• Info — deploy/mac-production/README.md:102: de procedure voor een gewijzigde launcher benoemt de niet-rollbackbare stap en de noodzakelijke service-herstart.
• Info — docs/handleiding.md:84: de Git-beperkingen van de servicegebruiker zijn duidelijk vastgelegd.

Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden.

Tests: niet uitgevoerd; deze PR wijzigt uitsluitend documentatie.

## Verdict: APPROVED Geen blokkerende findings. - Info — `deploy/mac-production/README.md:66`: het runbook gebruikt nu een macOS-compatibele schrijfroute voor `repo-paths` en herstelt expliciet eigenaar en permissies. - Info — `deploy/mac-production/README.md:102`: de procedure voor een gewijzigde launcher benoemt de niet-rollbackbare stap en de noodzakelijke service-herstart. - Info — `docs/handleiding.md:84`: de Git-beperkingen van de servicegebruiker zijn duidelijk vastgelegd. Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. Tests: niet uitgevoerd; deze PR wijzigt uitsluitend documentatie.

janpeter added 1 commit 2026-09-28 18:15:02 +02:00

Merge remote-tracking branch 'origin/main' into docs/repo-paths-command ... 8268996fc1

# Conflicts:
#	docs/handleiding.md

s4m-codex-reviewer approved these changes 2026-09-28 18:15:56 +02:00

s4m-codex-reviewer left a comment

Copy link

Verdict: APPROVED

Findings

• Geen findings met blocker- of error-severity vastgesteld. De gewijzigde runbook-instructies gebruiken een root-shell voor het wegschrijven van de piped inhoud en beperken daarna eigenaar en permissies; de launcher-procedure beschrijft een expliciete build en herstart. Documentatie is consistent met de gewijzigde gebruiksbeperking van Git.

Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden.

Tests: niet van toepassing; deze PR wijzigt uitsluitend documentatie.

# Verdict: APPROVED ## Findings - Geen findings met blocker- of error-severity vastgesteld. De gewijzigde runbook-instructies gebruiken een root-shell voor het wegschrijven van de piped inhoud en beperken daarna eigenaar en permissies; de launcher-procedure beschrijft een expliciete build en herstart. Documentatie is consistent met de gewijzigde gebruiksbeperking van Git. Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. Tests: niet van toepassing; deze PR wijzigt uitsluitend documentatie.

janpeter merged commit 8b95e81a5a into main 2026-09-28 19:03:59 +02:00

janpeter referenced this pull request from a commit 2026-09-28 19:04:01 +02:00

Merge pull request 'docs(runbook): repo-paths aanmaken zonder install /dev/stdin' (#266) from docs/repo-paths-command into main

Sign in to join this conversation.

Reviewers

No reviewers

s4m-codex-reviewer

Labels

Clear labels   

severity/s2

  

severity/s3

No labels

severity/s2

severity/s3

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

janpeter/Ops-dashboard!266

No description provided.