Skip to content

Fix artifact upload: include hidden files for .claude-issues/#1068

Merged
dpark01 merged 1 commit into
mainfrom
dp/fix-artifact-hidden-files
May 11, 2026
Merged

Fix artifact upload: include hidden files for .claude-issues/#1068
dpark01 merged 1 commit into
mainfrom
dp/fix-artifact-hidden-files

Conversation

@dpark01
Copy link
Copy Markdown
Member

@dpark01 dpark01 commented May 11, 2026

Summary

  • upload-artifact@v6 excludes hidden directories (names starting with .) by default since Sep 2024
  • Since .claude-issues/ is a dot-directory, all files inside it were silently skipped during artifact upload, causing the "No files were found" warning
  • Adding include-hidden-files: true fixes this

This is the root cause of the artifact upload warning that persisted through PRs #1064 and #1067 — the glob pattern was never the issue; it was the hidden-directory exclusion.

Test plan

  • Merge and trigger workflow_dispatch with test_cve_id=CVE-2026-42246 and dry_run=true
  • Verify artifact upload succeeds (no "No files found" warning)
  • Verify claude-cve-analysis artifact contains the .md file

upload-artifact@v6 excludes hidden directories (starting with .)
by default since Sep 2024. Since .claude-issues/ is a dot-directory,
all files were silently skipped. Adding include-hidden-files: true
fixes the "No files were found" warning.
Copilot AI review requested due to automatic review settings May 11, 2026 17:58
@dpark01 dpark01 enabled auto-merge (squash) May 11, 2026 17:58
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Fixes the actions/upload-artifact@v6 upload warning for the Claude CVE analysis artifact by explicitly including files under the hidden .claude-issues/ directory.

Changes:

  • Set include-hidden-files: true on the claude-cve-analysis artifact upload step so .claude-issues/*.md is not silently skipped.

@dpark01 dpark01 merged commit 64c5c48 into main May 11, 2026
60 of 66 checks passed
@dpark01 dpark01 deleted the dp/fix-artifact-hidden-files branch May 11, 2026 18:02
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