Skip to content

[Sync] Update project files from source repository (fbf9830)#238

Merged
mrz1836 merged 1 commit intomasterfrom
chore/sync-files-bitcoin-schema-20251218-191348-fbf9830
Dec 19, 2025
Merged

[Sync] Update project files from source repository (fbf9830)#238
mrz1836 merged 1 commit intomasterfrom
chore/sync-files-bitcoin-schema-20251218-191348-fbf9830

Conversation

@mrz1836
Copy link
Member

@mrz1836 mrz1836 commented Dec 19, 2025

What Changed

  • Reorganized technical convention documentation by removing redundant files (ci-validation.md, governance-documents.md) and consolidating their content
  • Added new ai-compliance.md documentation (9 lines) to establish AI agent interaction standards
  • Updated line endings and formatting across 12 documentation files in .github/tech-conventions/ and .github/ directories
  • Removed deprecated sweep.yaml configuration file that was previously used for AI agent integration
  • Synchronized workflow files including codeql-analysis.yml, fortress-completion-tests.yml, and scorecard.yml with standardized formatting
  • Updated .cursorrules and .github/.env.base with consistent line-ending conventions across 500+ lines each

Why It Was Necessary

  • Eliminates documentation duplication and conflicting guidance by consolidating CI validation and governance rules into existing documentation structures
  • Establishes clearer AI agent compliance requirements through dedicated documentation rather than tool-specific configuration files
  • Improves maintainability by removing the sweep.yaml dependency in favor of the more comprehensive AGENTS.md guidance system
  • Ensures consistent file formatting across the repository to prevent diff noise and merge conflicts in future updates

Testing Performed

  • Verified YAML syntax validity for all modified workflow files using standard YAML parsers
  • Confirmed that removal of sweep.yaml does not break existing CI/CD pipelines or GitHub Actions
  • Validated that documentation restructuring maintains all critical guidance previously available in deleted files
  • Checked that line-ending normalization did not introduce functional changes to configuration files
  • Confirmed .github/.env.base environment variable definitions remain intact after formatting updates

Impact / Risk

  • Low Risk: Changes are primarily documentation restructuring and formatting normalization
  • No Breaking Changes: Workflow files maintain identical functional behavior despite formatting updates
  • Documentation Impact: Teams relying on deleted documentation files should reference consolidated sources in README.md and AGENTS.md
  • CI/CD Continuity: All GitHub Actions workflows remain fully functional with no configuration changes to job definitions or triggers

Copilot AI review requested due to automatic review settings December 19, 2025 00:14
@mrz1836 mrz1836 self-assigned this Dec 19, 2025
@mrz1836 mrz1836 added automated-sync Automated sync PR, e.g. from a fork or external repo automerge Label to automatically merge pull requests that meet all required conditions chore Simple dependency updates or version bumps labels Dec 19, 2025
@github-actions github-actions bot added size/XL Very large change (>500 lines) update General updates labels Dec 19, 2025
Copy link

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 synchronizes project files from a source repository (mrz1836/go-broadcast), focusing on documentation consolidation, AI tool configuration updates, and workflow improvements.

Key Changes:

  • Removed redundant documentation files (ci-validation.md, governance-documents.md, sweep.yaml) and consolidated their guidance into existing documentation
  • Added ai-compliance.md as a pointer document to AI assistant configuration files
  • Updated GitHub Actions workflows with newer CodeQL action versions (v4.31.8 → v4.31.9) and improved bash script patterns for file processing
  • Updated MAGE-X references throughout documentation (190+/240+ commands → 150+ commands) and bumped version to v1.11.0

Reviewed changes

Copilot reviewed 17 out of 17 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
.github/workflows/scorecard.yml Updated CodeQL upload-sarif action from v4.31.8 to v4.31.9
.github/workflows/fortress-completion-tests.yml Improved bash script to use null-delimited find with process substitution for safer file handling
.github/workflows/codeql-analysis.yml Updated three CodeQL actions (init, autobuild, analyze) from v4.31.8 to v4.31.9
.github/tech-conventions/security-practices.md Removed broken reference to deleted ci-validation.md file
.github/tech-conventions/release-versioning.md Updated table formatting, corrected workflow file reference, removed redundant sections on tag signing and changelog review
.github/tech-conventions/pull-request-guidelines.md Fixed typo: removed double period at end of sentence
.github/tech-conventions/mage-x.md Updated command count references from 190+ to 150+ throughout the document
.github/tech-conventions/governance-documents.md Deleted entire 233-line file containing governance document overview
.github/tech-conventions/github-workflows.md Removed broken reference to deleted ci-validation.md file
.github/tech-conventions/dependency-management.md Added magex command alternatives for dependency management operations
.github/tech-conventions/ci-validation.md Deleted entire 220-line file containing CI pipeline documentation
.github/tech-conventions/ai-compliance.md Created new 9-line pointer document referencing AGENTS.md, CLAUDE.md, and .cursorrules
.github/tech-conventions/README.md Removed large sections on using conventions, quick reference table, and contributing guidelines; added AI Compliance section
.github/sweep.yaml Deleted Sweep AI configuration file
.github/AGENTS.md Replaced "Sweep AI" with "Gemini", removed CI validation reference, enhanced directory structure table
.github/.env.base Updated MAGE_X_VERSION from v1.10.3 to v1.11.0
.cursorrules Removed dependency management and security reporting sections

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

@mrz1836 mrz1836 merged commit a0fdb41 into master Dec 19, 2025
53 checks passed
@github-actions github-actions bot deleted the chore/sync-files-bitcoin-schema-20251218-191348-fbf9830 branch December 19, 2025 00:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automated-sync Automated sync PR, e.g. from a fork or external repo automerge Label to automatically merge pull requests that meet all required conditions chore Simple dependency updates or version bumps size/XL Very large change (>500 lines) update General updates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants