Skip to content

Add frontmatter hash comparison logging to workflow outdated check #13132

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
pelikhan merged 3 commits into from
Feb 1, 2026
Merged

Add frontmatter hash comparison logging to workflow outdated check #13132

pelikhan merged 3 commits into from
Feb 1, 2026
+360 −0

Conversation

Copy link
Contributor

Copilot AI commented Feb 1, 2026 •
edited
Loading

Plan: Add Frontmatter Hash Logging to Outdated Workflow Check

  • Understand the current check_workflow_timestamp_api.cjs implementation
  • Import frontmatter hash utilities from frontmatter_hash_pure.cjs
  • Fetch lock file content using GitHub API to extract existing hash
  • Fetch source .md file content using GitHub API to compute new hash
  • Add logging to display both hashes when checking if workflow is outdated
  • Ensure no changes to current logic (only add logging)
  • Add appropriate tests
  • Run tests and validate changes
  • Refactor GitHub helper into separate file per review feedback
Original prompt

Add front matter hash computation and display to the log in the JavaScript that check if workflows are outdated (do not change current logic)

  • use GitHub getFileContent
  • show lock file frontmatter hash vs recomputed hash

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

@pelikhan
Add frontmatter hash comparison logging to workflow outdated check
d1775de 
- Import computeFrontmatterHash, extractHashFromLockFile, and createGitHubFileReader from frontmatter_hash_pure.cjs
- Add getFileContent helper to fetch file content from GitHub API
- Add logFrontmatterHashComparison to extract hash from lock file and compute new hash
- Display both hashes in logs when checking if workflow is outdated
- Add tests for hash comparison, missing hash, and error handling
- All tests passing

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Add front matter hash computation and display in logs Add frontmatter hash comparison logging to workflow outdated check Feb 1, 2026
Copilot AI requested a review from pelikhan February 1, 2026 16:11
pelikhan
pelikhan reviewed Feb 1, 2026
View reviewed changes
actions/setup/js/check_workflow_timestamp_api.cjs Outdated
}

// Helper function to get file content using GitHub API
async function getFileContent(path) {
Copy link
Contributor

@pelikhan pelikhan Feb 1, 2026

Choose a reason for hiding this comment

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

@copilot refactor github helper file

Copy link
Contributor Author

Copilot AI Feb 1, 2026

Choose a reason for hiding this comment

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

Refactored getFileContent into a new github_api_helpers.cjs file with comprehensive test coverage. The helper is now reusable across the codebase. (da6de2a)

@pelikhan
Refactor: Extract getFileContent to github_api_helpers
da6de2a 
- Create github_api_helpers.cjs with reusable getFileContent function
- Add comprehensive test coverage for github_api_helpers
- Update check_workflow_timestamp_api.cjs to use the new helper
- All tests passing

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
@pelikhan pelikhan added the smoke label Feb 1, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Feb 1, 2026 •
edited by agentic-workflows-dev bot
Loading

🎉 Yo ho ho! Changeset Generator found the treasure and completed successfully! ⚓💰

@github-actions
Copy link
Contributor

github-actions bot commented Feb 1, 2026

📰 BREAKING: Smoke Copilot is now investigating this pull request. Sources say the story is developing...

@github-actions
Copy link
Contributor

github-actions bot commented Feb 1, 2026 •
edited
Loading

✨ The prophecy is fulfilled... Smoke Codex has completed its mystical journey. The stars align. 🌟

@github-actions
Copy link
Contributor

github-actions bot commented Feb 1, 2026 •
edited
Loading

🎬 THE END — Smoke Claude MISSION: ACCOMPLISHED! The hero saves the day! ✨

Copilot AI requested a review from pelikhan February 1, 2026 16:19
@github-actions
Copy link
Contributor

github-actions bot commented Feb 1, 2026

Agent Container Tool Check

Tool Status Version
bash ✅ 5.2.21
sh ✅ available
git ✅ 2.52.0
jq ✅ 1.7
yq ✅ 4.50.1
curl ✅ 8.5.0
gh ✅ 2.86.0
node ✅ 24.13.0
python3 ✅ 3.13.11
go ✅ 1.24.12
java ❌ not found
dotnet ❌ not found

Result: 10/12 tools available ⚠️

Missing tools: java, dotnet

AI generated by Agent Container Smoke Test

@github-actions github-actions bot removed the smoke label Feb 1, 2026
@github-actions github-actions bot mentioned this pull request Feb 1, 2026
Closed
@github-actions
Copy link
Contributor

github-actions bot commented Feb 1, 2026

Smoke test results
PRs: Remove copy-project safe output; [jsweep] Clean campaign_labels.cjs
GitHub MCP merged PRs ✅
safeinputs gh pr list ✅
Serena activate ✅
Playwright github title ✅
Tavily search ✅
File write+cat ✅
make build ✅
Overall: PASS

AI generated by Smoke Codex

@github-actions
Copy link
Contributor

github-actions bot commented Feb 1, 2026

Smoke Test Results ✅

PRs Tested:

Tests:
✅ GitHub MCP
✅ Safe Inputs GH CLI
✅ Serena MCP
✅ Playwright
✅ File Writing
✅ Bash Tool
✅ Discussion Interaction
✅ Build gh-aw

Overall Status: PASS

cc @pelikhan

AI generated by Smoke Copilot

@github-actions
Copy link
Contributor

github-actions bot commented Feb 1, 2026

📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤

@pelikhan pelikhan marked this pull request as ready for review February 1, 2026 16:23
@pelikhan pelikhan merged commit 04fd063 into main Feb 1, 2026
122 checks passed
@pelikhan pelikhan deleted the copilot/add-frontmatter-hash-computation branch February 1, 2026 16:23
@github-actions github-actions bot mentioned this pull request Feb 1, 2026
Closed
@github-actions
Copy link
Contributor

github-actions bot commented Feb 1, 2026

Smoke Test: Claude - PASS ✅

PRs Reviewed:

Test Results:
✅ GitHub MCP
✅ Safe Inputs GH CLI
✅ Serena MCP
✅ Make Build
✅ Playwright
✅ Tavily
✅ File Writing
✅ Bash
✅ Discussion Interaction

Overall Status: PASS

AI generated by Smoke Claude

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pelikhan pelikhan Awaiting requested review from pelikhan

Assignees

@pelikhan pelikhan

Copilot code review Copilot

Labels

smoke-codex smoke-copilot

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@pelikhan