Skip to content

Rename frontmatter field fingerprint to campaign#3737

Merged
mnkiefer merged 5 commits into
mainfrom
copilot/rename-fingerprint-to-campaign
Nov 12, 2025
Merged

Rename frontmatter field fingerprint to campaign#3737
mnkiefer merged 5 commits into
mainfrom
copilot/rename-fingerprint-to-campaign

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Nov 12, 2025

Renames the fingerprint frontmatter field to campaign throughout the codebase for improved semantic clarity.

Changes

Schema & Types

  • Updated JSON schema: fingerprintcampaign
  • Renamed struct field in WorkflowData
  • Renamed Go functions: extractFingerprint()extractCampaign(), buildWorkflowMetadataEnvVarsWithFingerprint()buildWorkflowMetadataEnvVarsWithCampaign()

JavaScript/CJS Files

  • Renamed get_fingerprint.cjsget_campaign.cjs
  • Updated function: getFingerprint()getCampaign()
  • Updated all require() statements in safe output scripts

Environment & Output Format

  • Environment variable: GH_AW_FINGERPRINTGH_AW_CAMPAIGN
  • HTML comment format: <!-- fingerprint: value --><!-- campaign: value -->

Tests & Documentation

  • Renamed test files and updated all test cases
  • Updated reference documentation and workflow examples

Example

---
on: workflow_dispatch
campaign: team-alpha-2024  # Previously: fingerprint: team-alpha-2024
safe-outputs:
  create-issue:
---

Compiled workflows now expose GH_AW_CAMPAIGN and inject <!-- campaign: team-alpha-2024 --> into created assets.

Breaking Change

⚠️ Workflows using fingerprint: must update to campaign:. Validation rules (min 8 chars, alphanumeric + hyphens/underscores) remain unchanged.

Original prompt

Rename frontmatter field fingerprint to campaign.


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

Copilot AI and others added 3 commits November 12, 2025 16:08
Co-authored-by: mnkiefer <8320933+mnkiefer@users.noreply.github.com>
Co-authored-by: mnkiefer <8320933+mnkiefer@users.noreply.github.com>
Co-authored-by: mnkiefer <8320933+mnkiefer@users.noreply.github.com>
Copilot AI changed the title [WIP] Rename frontmatter field fingerprint to campaign Rename frontmatter field fingerprint to campaign Nov 12, 2025
Copilot AI requested a review from mnkiefer November 12, 2025 16:40
@mnkiefer mnkiefer marked this pull request as ready for review November 12, 2025 17:09
Copilot AI review requested due to automatic review settings November 12, 2025 17:09
@mnkiefer mnkiefer merged commit c7b45ca into main Nov 12, 2025
15 checks passed
@mnkiefer mnkiefer deleted the copilot/rename-fingerprint-to-campaign branch November 12, 2025 17:10
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

This PR renames the fingerprint frontmatter field to campaign throughout the codebase for improved semantic clarity. The change is comprehensive and systematic, touching all relevant components including schemas, Go code, JavaScript modules, tests, documentation, and workflow examples.

Key Changes:

  • Renamed frontmatter field from fingerprint to campaign with consistent validation rules (8+ chars, alphanumeric + hyphens/underscores)
  • Updated environment variable from GH_AW_FINGERPRINT to GH_AW_CAMPAIGN
  • Changed HTML comment format from <!-- fingerprint: value --> to <!-- campaign: value -->

Reviewed Changes

Copilot reviewed 93 out of 93 changed files in this pull request and generated no comments.

Show a summary per file
File Description
pkg/workflow/safe_outputs.go Renamed helper functions and environment variable references
pkg/workflow/compiler.go Updated WorkflowData struct field and extraction logic
pkg/workflow/frontmatter_extraction.go Renamed extraction function and error messages
pkg/workflow/js/get_campaign.cjs New JavaScript module (renamed from get_fingerprint.cjs)
pkg/workflow/js/get_campaign.test.cjs New test file with updated test cases
pkg/workflow/js/*.cjs Updated require statements in safe output scripts
pkg/workflow/js.go Updated embedded script variable names
pkg/workflow/campaign_test.go New test file (renamed from fingerprint_test.go)
pkg/workflow/campaign_integration_test.go Updated integration tests
pkg/parser/schemas/main_workflow_schema.json Updated JSON schema property name and description
docs/src/content/docs/reference/frontmatter-full.md Updated documentation
.github/workflows/*.md Updated workflow examples to use new field name
.github/workflows/*.lock.yml Regenerated compiled workflows with new env vars and functions

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

3 participants