Skip to content

Create a build script for all functions in a consolidated PSM1 file#1586

Merged
SamErde merged 14 commits intomainfrom
BuildMergedPSM1
Apr 14, 2026
Merged

Create a build script for all functions in a consolidated PSM1 file#1586
SamErde merged 14 commits intomainfrom
BuildMergedPSM1

Conversation

@SamErde
Copy link
Copy Markdown
Contributor

@SamErde SamErde commented Apr 6, 2026

Description

This pull request adds a new Maester PowerShell module build process.

Consolidates all source files from powershell/internal/ and powershell/public/ into a single Maester.psm1, consolidates ORCA class definitions into OrcaClasses.ps1, auto-generates the FunctionsToExport list via AST parsing, and copies static assets and tests into the /module directory with the manifest and custom formatting files. The source tree is never modified.

Completes #1561, #1562, #1563, #1565

Contribution Checklist

Before submitting this PR, please confirm you have completed the following:

  • 📖 Read the guidelines for contributing to this repository.
  • 🧪 Ensure the build and unit tests pass by running /powershell/tests/pester.ps1 on your local system.

@SamErde SamErde self-assigned this Apr 6, 2026
@SamErde SamErde added enhancement New feature or request maester-test Related to a Maester test maester-core Related to the core Maester framework labels Apr 6, 2026
@codacy-production
Copy link
Copy Markdown

codacy-production Bot commented Apr 6, 2026

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

TIP This summary will be updated as you push new changes. Give us feedback

@SamErde SamErde marked this pull request as ready for review April 6, 2026 18:00
@SamErde SamErde requested a review from a team as a code owner April 6, 2026 18:01
Copilot AI review requested due to automatic review settings April 6, 2026 18:01
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

Adds a new build process to generate a publishable Maester module artifact under ./module/ without modifying the source tree, focusing on faster module import via consolidation.

Changes:

  • Adds build/Build-MaesterModule.ps1 to AST-discover public exports, consolidate Maester.psm1, consolidate ORCA classes, update the manifest, and copy assets/tests.
  • Adds build/Build-MaesterModule.md documenting usage and build phases.
  • Ignores generated build output (/module/) and updates the optimization plan formatting.

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 6 comments.

File Description
build/maester-optimization-plan.md Minor markdown formatting adjustments to the optimization plan.
build/Build-MaesterModule.ps1 New end-to-end build script for producing a consolidated module artifact and updated manifest.
build/Build-MaesterModule.md Documentation for the new build script, phases, and output structure.
.gitignore Ensures /module/ build output is not committed.

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

Comment thread build/Build-MaesterModule.ps1 Outdated
Comment thread build/Build-MaesterModule.ps1 Outdated
Comment thread build/Build-MaesterModule.ps1
Comment thread build/Build-MaesterModule.ps1 Outdated
Comment thread build/Build-MaesterModule.ps1
Comment thread build/Build-MaesterModule.md Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented Apr 6, 2026

Deploying maester with  Cloudflare Pages  Cloudflare Pages

Latest commit: cfa7d5c
Status:🚫  Build failed.

View logs

Comment thread build/Build-MaesterModule.ps1
@SamErde SamErde merged commit 4f9ce75 into main Apr 14, 2026
7 of 8 checks passed
@SamErde SamErde deleted the BuildMergedPSM1 branch April 14, 2026 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request maester-core Related to the core Maester framework maester-test Related to a Maester test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants