Skip to content

Use slug-safe workflow names for log filenames and update harness template#311

Merged
tbrandenburg merged 1 commit intomainfrom
codex/verify-log-file-naming-consistency
Mar 25, 2026
Merged

Use slug-safe workflow names for log filenames and update harness template#311
tbrandenburg merged 1 commit intomainfrom
codex/verify-log-file-naming-consistency

Conversation

@tbrandenburg
Copy link
Copy Markdown
Owner

Motivation

  • Ensure log filenames are slug-safe, deterministic, and include UTC timestamp and PID to avoid collisions and permission issues.
  • Make the prompt/template documentation consistent about the required filename format using bracketed placeholders.

Description

  • Add WORKFLOW_NAME, WORKFLOW_SLUG, LOG_TIMESTAMP, and LOG_BASENAME variables to the Bash harness template to build a made-[workflow-name]-[timestamp]-[PID].log filename safely using tr and sed for slugification and date -u for UTC timestamps.
  • Use LOG_BASENAME when selecting the log file path for /var/log or the fallback /tmp/made-harness-logs, and ensure the fallback directory is created with mkdir -p.
  • Update the template documentation examples to use bracketed placeholder notation (e.g. /var/log/made-[workflow-name]-[timestamp]-[PID].log).
  • Preserve existing logging behavior of emitting ISO-8601 UTC timestamps, writing to stderr, and appending to the log file when writable.

Testing

  • Ran the repository linters and the frontend test suite against the modified templates and harness script; all checks passed.

Codex Task

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 25, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
made Ready Ready Preview, Comment Mar 25, 2026 6:58pm

@tbrandenburg tbrandenburg merged commit e29a300 into main Mar 25, 2026
6 of 7 checks passed
@tbrandenburg tbrandenburg deleted the codex/verify-log-file-naming-consistency branch March 25, 2026 18:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant