Skip to content

Fix jqschema.sh: ensure /tmp/gh-aw directory exists before writing#1957

Merged
pelikhan merged 3 commits intomainfrom
copilot/prepare-jqschema-moxie
Oct 18, 2025
Merged

Fix jqschema.sh: ensure /tmp/gh-aw directory exists before writing#1957
pelikhan merged 3 commits intomainfrom
copilot/prepare-jqschema-moxie

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Oct 18, 2025

Problem

The .github/workflows/shared/jqschema.md workflow file was attempting to create /tmp/gh-aw/jqschema.sh without first ensuring the parent directory exists. This would cause workflow failures when the /tmp/gh-aw/ directory doesn't exist:

cat > /tmp/gh-aw/jqschema.sh << 'EOF'
# Error: No such file or directory

Solution

Added mkdir -p /tmp/gh-aw before the cat command to ensure the directory exists, following the established pattern used throughout the codebase (e.g., in shared/genaiscript.md, shared/ffmpeg.md, shared/mcp/drain3.md):

mkdir -p /tmp/gh-aw
cat > /tmp/gh-aw/jqschema.sh << 'EOF'
#!/usr/bin/env bash
# jqschema.sh
...

Changes

  • Modified .github/workflows/shared/jqschema.md to create the directory before writing to it
  • Recompiled all affected workflows using make recompile, updating 6 .lock.yml files

All workflows that import shared/jqschema.md now have the directory creation step properly included in their compiled output.

Original prompt

Make sure jqschema moxie the /tmp/gh-aw/ folder before cat into it.
Recompile


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Ensure jqschema moxie in /tmp/gh-aw/ before concatenation Fix jqschema.sh: ensure /tmp/gh-aw directory exists before writing Oct 18, 2025
Copilot AI requested a review from pelikhan October 18, 2025 18:47
@pelikhan pelikhan marked this pull request as ready for review October 18, 2025 18:48
@github-actions
Copy link
Copy Markdown
Contributor

Agentic Changeset Generator triggered by this pull request.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@pelikhan pelikhan merged commit 1682c0e into main Oct 18, 2025
4 checks passed
@pelikhan pelikhan deleted the copilot/prepare-jqschema-moxie branch October 18, 2025 19:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants