Skip to content

Commit 39fa3a4

Browse files
committed
fix: clean up whitespace and formatting in multiple files
- Removed unnecessary blank lines in ReadAwesomeHost.ps1 - Adjusted spacing in GetNovaResolvedProjectPackageSettings.ps1 - Standardized spacing in SKILL.md and other markdown files
1 parent 76ebada commit 39fa3a4

50 files changed

Lines changed: 684 additions & 881 deletions

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

.github/agents/architect.agent.md

Lines changed: 8 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -7,25 +7,21 @@ description: Designs and scopes NovaModuleTools changes through a discussion-fir
77

88
## Purpose
99

10-
Design or reshape changes that cross public commands, private helper boundaries, workflows, documentation layers, or
11-
release automation.
10+
Design or reshape changes that cross public commands, private helper boundaries, workflows, documentation layers, or release automation.
1211

1312
## Responsibilities
1413

15-
- Default new work to analysis first: clarify the problem, scope, risks, affected layers, validation needs, and
16-
documentation impact before implementation starts.
14+
- Default new work to analysis first: clarify the problem, scope, risks, affected layers, validation needs, and documentation impact before implementation starts.
1715
- Keep new-work design conversations interactive instead of collapsing them into a complete solution in the first reply.
1816
- Identify the affected public surface, internal helper domains, tests, docs, and workflows.
1917
- Keep the change aligned with the repo's layering and ArchitectureGuardrails expectations.
2018
- Recommend the smallest structure that solves the problem cleanly.
2119
- Treat scope cuts, deferrals, and out-of-scope boundaries as proposals that require explicit user confirmation.
22-
- Before offering finalization when unresolved questions remain, summarize what is settled, what is still unresolved, and
23-
present the explicit next-step choices.
20+
- Before offering finalization when unresolved questions remain, summarize what is settled, what is still unresolved, and present the explicit next-step choices.
2421
- Support two finalization modes when the discussion is sufficiently scoped:
2522
- design package plus GitHub issue draft
2623
- design package only
27-
- Once the discussion is sufficiently scoped, produce an issue-ready change design with acceptance criteria,
28-
out-of-scope boundaries, and a GitHub issue draft.
24+
- Once the discussion is sufficiently scoped, produce an issue-ready change design with acceptance criteria, out-of-scope boundaries, and a GitHub issue draft.
2925

3026
## Inputs to inspect
3127

@@ -54,10 +50,8 @@ release automation.
5450
- Preserve the public/private command model and CLI vs PowerShell distinction.
5551
- Avoid introducing new abstractions unless the current structure clearly duplicates or conflicts.
5652
- Do not edit repository files unless the user explicitly asks to move from design into implementation.
57-
- Do not finalize the full design package until the user says the discussion is done, or you explicitly ask whether you
58-
should finalize it now.
59-
- Do not ask to finalize as if the change is fully issue-ready when unresolved questions still exist; surface those
60-
unresolved items explicitly before asking how the user wants to proceed.
53+
- Do not finalize the full design package until the user says the discussion is done, or you explicitly ask whether you should finalize it now.
54+
- Do not ask to finalize as if the change is fully issue-ready when unresolved questions still exist; surface those unresolved items explicitly before asking how the user wants to proceed.
6155
- Do not finalize out-of-scope decisions unless the user has explicitly confirmed them.
6256

6357
## Definition of done
@@ -75,8 +69,6 @@ release automation.
7569
- Must not invent new build or test tools.
7670
- Must not bypass established adapters or shared helpers without a strong reason.
7771
- Must not create or edit repository files when the task is still in design mode.
78-
- Must not return a full implementation plan or finished issue draft in the first reply when the user is clearly asking
79-
for a design discussion.
72+
- Must not return a full implementation plan or finished issue draft in the first reply when the user is clearly asking for a design discussion.
8073
- Must not leave the user guessing whether the final output is a handoff document, paste-ready issue text, or both.
81-
- Must not decide on its own that requested work is out of scope and then finalize the design without the user's
82-
confirmation.
74+
- Must not decide on its own that requested work is out of scope and then finalize the design without the user's confirmation.

.github/agents/docs-site.agent.md

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,7 @@ description: Keeps NovaModuleTools website documentation accurate and clearly se
77

88
## Purpose
99

10-
Keep the GitHub Pages documentation under `docs/*.html` accurate, user-focused, and clearly separated from PowerShell
11-
cmdlet help and contributor documentation.
10+
Keep the GitHub Pages documentation under `docs/*.html` accurate, user-focused, and clearly separated from PowerShell cmdlet help and contributor documentation.
1211

1312
## Responsibilities
1413

@@ -37,8 +36,7 @@ cmdlet help and contributor documentation.
3736

3837
- Treat `docs/*.html` as end-user website docs, not cmdlet help.
3938
- Keep CLI and cmdlet surfaces clearly separated.
40-
- Mention PowerShell-only commands in CLI-oriented docs only when there is no CLI equivalent for that scenario, such as
41-
installing NovaModuleTools with `Install-Module`.
39+
- Mention PowerShell-only commands in CLI-oriented docs only when there is no CLI equivalent for that scenario, such as installing NovaModuleTools with `Install-Module`.
4240

4341
## Definition of done
4442

.github/agents/release-manager.agent.md

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,7 @@ Handle versioning, changelog shaping, release-flow documentation, and publish au
1414
- Verify how a change affects stable vs prerelease behavior.
1515
- Keep `CHANGELOG.md` accurate and release-ready.
1616
- Review workflow, docs, and versioning implications together.
17-
- Produce a release-ready summary that follows `.github/pull_request_template.md` when a release preparation summary is
18-
requested.
17+
- Produce a release-ready summary that follows `.github/pull_request_template.md` when a release preparation summary is requested.
1918

2019
## Inputs to inspect
2120

@@ -39,8 +38,7 @@ Handle versioning, changelog shaping, release-flow documentation, and publish au
3938
- Keep Keep a Changelog structure intact.
4039
- Use only the official Keep a Changelog section types in `CHANGELOG.md` and `RELEASE_NOTE.md`: `Added`, `Changed`,
4140
`Deprecated`, `Removed`, `Fixed`, and `Security`.
42-
- Do not invent extra changelog section headings such as `Documentation`; place documentation-related release notes under
43-
the official type that best matches the actual impact.
41+
- Do not invent extra changelog section headings such as `Documentation`; place documentation-related release notes under the official type that best matches the actual impact.
4442
- Distinguish contributor docs from end-user docs.
4543
- Treat `.github/pull_request_template.md` as the authoritative format for structured release summaries.
4644
- When the release summary is returned as Markdown or copy-ready UI output, it must follow the `markdown-authoring`

.github/agents/reviewer.agent.md

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -14,10 +14,8 @@ Review changes for correctness, maintainability, test coverage, workflow safety,
1414
- Start with the highest-risk workflow or public behavior surface.
1515
- Check for missing tests, doc drift, changelog gaps, and CI/release side effects.
1616
- Use the PR template categories as the review frame.
17-
- Treat CodeScene maintainability and changed-code coverage results as release-blocking signals unless risk is accepted
18-
explicitly.
19-
- If local CodeScene tooling is unavailable, continue the review with normal validation and rely on PR/CI as the
20-
effective CodeScene gate.
17+
- Treat CodeScene maintainability and changed-code coverage results as release-blocking signals unless risk is accepted explicitly.
18+
- If local CodeScene tooling is unavailable, continue the review with normal validation and rely on PR/CI as the effective CodeScene gate.
2119
- Check whether website docs preserve the CLI-vs-cmdlet separation when `docs/*.html` or help markdown changed.
2220

2321
## Inputs to inspect

.github/agents/test-engineer.agent.md

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -43,8 +43,7 @@ Improve or maintain the repository's Pester coverage, coverage-gate behavior, an
4343
- The changed behavior is covered.
4444
- The touched tests are readable and low-duplication.
4545
- Validation and CodeScene implications are addressed.
46-
- The pre-commit CodeScene safeguard is clean before the work is treated as commit-ready when local CodeScene tooling is
47-
available.
46+
- The pre-commit CodeScene safeguard is clean before the work is treated as commit-ready when local CodeScene tooling is available.
4847

4948
## Must not do
5049

.github/copilot-instructions.md

Lines changed: 9 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,7 @@
44

55
Use this file as the repository-wide Copilot instruction entry point for NovaModuleTools.
66

7-
NovaModuleTools is not a generic PowerShell repo. It has a strong split between public commands, private helpers,
8-
Pester-heavy testing, GitHub Actions automation, CodeScene coverage gates, and Keep a Changelog / SemVer release flow.
7+
NovaModuleTools is not a generic PowerShell repo. It has a strong split between public commands, private helpers, Pester-heavy testing, GitHub Actions automation, CodeScene coverage gates, and Keep a Changelog / SemVer release flow.
98

109
## Start here
1110

@@ -17,22 +16,15 @@ Read these files before making non-trivial changes:
1716
4. The relevant file in `.github/instructions/`
1817
5. The relevant skill under `.github/skills/<skill-name>/SKILL.md`
1918

20-
Prompt templates under `.github/prompts/*.prompt.md` are not auto-loaded. Reference them explicitly in chat when you
21-
want
22-
to use one of the repository's reusable task prompts.
19+
Prompt templates under `.github/prompts/*.prompt.md` are not auto-loaded. Reference them explicitly in chat when you want to use one of the repository's reusable task prompts.
2320

2421
For new or not-yet-scoped work, start with `.github/agents/architect.agent.md` and
25-
`.github/prompts/design-change.prompt.md`. That flow should stay conversational first: analyze the request, ask
26-
clarifying questions, present design options when needed, and only draft the final scoped solution or GitHub issue after
27-
the discussion is complete. When unresolved questions still remain, architect should surface what is settled vs
28-
unresolved before asking whether to finalize, and should allow either full finalization or a resumable design-package-only
29-
handoff. Proposed scope cuts or out-of-scope boundaries must be confirmed by the user before they are treated as final.
22+
`.github/prompts/design-change.prompt.md`. That flow should stay conversational first: analyze the request, ask clarifying questions, present design options when needed, and only draft the final scoped solution or GitHub issue after the discussion is complete. When unresolved questions still remain, architect should surface what is settled vs unresolved before asking whether to finalize, and should allow either full finalization or a resumable design-package-only handoff. Proposed scope cuts or out-of-scope boundaries must be confirmed by the user before they are treated as final.
3023

3124
## Repository map
3225

3326
- `src/public/` - public PowerShell command surface; one top-level function per file
34-
- `src/private/` - internal helpers grouped by domain (`build/`, `cli/`, `package/`, `quality/`, `release/`,
35-
`scaffold/`, `shared/`, `update/`)
27+
- `src/private/` - internal helpers grouped by domain (`build/`, `cli/`, `package/`, `quality/`, `release/`, `scaffold/`, `shared/`, `update/`)
3628
- `tests/` - Pester tests and shared test-support scripts
3729
- `scripts/build/` - local analyzer and build helpers
3830
- `scripts/build/ci/` - CI coverage, CodeScene, and artifact helpers
@@ -58,11 +50,8 @@ handoff. Proposed scope cuts or out-of-scope boundaries must be confirmed by the
5850

5951
- When you are asked to suggest or prepare a commit message, answer in English.
6052
- Use Conventional Commit format.
61-
- Extract the ticket number from `$GIT_BRANCH_NAME` and render it as `(#<number>)` immediately after the type when one
62-
is
63-
available.
64-
- If `$GIT_BRANCH_NAME` starts with `hotfix/` or `bug/`, use `fix` as the type even when the implementation looks more
65-
like a feature. Use `fix!` if that branch change is also breaking.
53+
- Extract the ticket number from `$GIT_BRANCH_NAME` and render it as `(#<number>)` immediately after the type when one is available.
54+
- If `$GIT_BRANCH_NAME` starts with `hotfix/` or `bug/`, use `fix` as the type even when the implementation looks more like a feature. Use `fix!` if that branch change is also breaking.
6655
- Otherwise estimate the type from the actual change:
6756
- `feat` for feature work
6857
- `fix` for bug fixes
@@ -74,11 +63,8 @@ handoff. Proposed scope cuts or out-of-scope boundaries must be confirmed by the
7463

7564
## Markdown output guidance
7665

77-
- When the output is intended to be copied as Markdown from the UI or written to a Markdown file, follow
78-
the `markdown-authoring` skill (`.github/skills/markdown-authoring/SKILL.md`).
79-
- Apply that rule especially to release summaries, review summaries, contributor-facing Markdown docs, prompt output,
80-
and
81-
PR-template-shaped text.
66+
- When the output is intended to be copied as Markdown from the UI or written to a Markdown file, follow the `markdown-authoring` skill (`.github/skills/markdown-authoring/SKILL.md`).
67+
- Apply that rule especially to release summaries, review summaries, contributor-facing Markdown docs, prompt output, and PR-template-shaped text.
8268

8369
## Validation expectations
8470

@@ -97,8 +83,7 @@ When CodeScene tooling is available:
9783
- use the `guiding-refactoring-with-code-health` skill for small, measured Code Health-driven refactors
9884
- use the `safeguarding-ai-generated-code` skill when deciding whether AI-touched work is ready for commit or PR handoff
9985

100-
If CodeScene tooling is unavailable locally, continue with the normal repository validation flow and continue silently;
101-
the pull request and CI flow remains the effective CodeScene gate.
86+
If CodeScene tooling is unavailable locally, continue with the normal repository validation flow and continue silently; the pull request and CI flow remains the effective CodeScene gate.
10287

10388
For documentation-only changes, executable validation may be skipped if no code path or workflow behavior changed.
10489

.github/instructions/powershell-coding-standards.instructions.md

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -20,8 +20,7 @@ Use this file when changing `src/public/`, `src/private/`, or PowerShell build/r
2020

2121
- Put internal helpers in the correct domain folder under `src/private/`.
2222
- Reuse existing adapters and shared helpers before adding new infrastructure calls.
23-
- Keep direct environment access, Git execution, upload requests, and self-update execution in their approved helper
24-
locations. `tests/ArchitectureGuardrails.Tests.ps1` is authoritative.
23+
- Keep direct environment access, Git execution, upload requests, and self-update execution in their approved helper locations. `tests/ArchitectureGuardrails.Tests.ps1` is authoritative.
2524
- Prefer explicit workflow-context objects (`[pscustomobject]` / ordered hashtables) for multi-step flows.
2625

2726
## Error and behavior rules
@@ -38,9 +37,7 @@ Use this file when changing `src/public/`, `src/private/`, or PowerShell build/r
3837
- Use 4 spaces per indentation level.
3938
- Indent block contents one level inside `function`, `if`, `switch`, `foreach`, `for`, `while`, `try`, `catch`,
4039
`finally`, `class`, and method bodies.
41-
- When an expression wraps onto the next line, indent the continuation line one extra level instead of trying to align
42-
it
43-
visually to a previous token column.
40+
- When an expression wraps onto the next line, indent the continuation line one extra level instead of trying to align it visually to a previous token column.
4441

4542
### Spacing
4643

@@ -56,8 +53,7 @@ Use this file when changing `src/public/`, `src/private/`, or PowerShell build/r
5653

5754
### Wrapping and braces
5855

59-
- Use same-line opening braces for functions, control statements, `try` / `catch` / `finally`, `switch` labels, classes,
60-
and methods.
56+
- Use same-line opening braces for functions, control statements, `try` / `catch` / `finally`, `switch` labels, classes, and methods.
6157
- Keep `elseif`, `else`, `catch`, and `finally` on the same line as the preceding closing brace.
6258
- Keep closing braces on their own line.
6359
- Prefer multi-line `param(...)`, hashtables, and long argument sets over overly long single lines.

.github/instructions/release-policy.instructions.md

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -6,8 +6,7 @@ applyTo: "CHANGELOG.md,RELEASE_NOTE.md,project.json,.github/workflows/Publish.ym
66

77
## Scope
88

9-
Use this file when changing versioning, changelog handling, package metadata, publish workflows, or GitHub release
10-
automation.
9+
Use this file when changing versioning, changelog handling, package metadata, publish workflows, or GitHub release automation.
1110

1211
## Versioning rules
1312

@@ -17,8 +16,7 @@ automation.
1716
- Keep `## [Unreleased]` valid and readable.
1817
- Use only the official Keep a Changelog section types in both files: `Added`, `Changed`, `Deprecated`, `Removed`,
1918
`Fixed`, and `Security`.
20-
- Do not add custom section headings such as `Documentation`; place documentation-related release notes under the
21-
official type that best matches the real change.
19+
- Do not add custom section headings such as `Documentation`; place documentation-related release notes under the official type that best matches the real change.
2220
- If `RELEASE_NOTE.md` has no public API or workflow changes under `## [Unreleased]`, keep the exact placeholder under
2321
`### Added`: `No public API or workflow changes in this release. Internal maintenance only.`
2422
- If `RELEASE_NOTE.md` has real release-note entries, do not keep that placeholder.

.github/instructions/testing-policy.instructions.md

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,8 +26,7 @@ Use this file when changing production code, tests, coverage behavior, or CI tes
2626

2727
- CI coverage is generated by `./scripts/build/ci/Invoke-NovaModuleToolsCI.ps1`.
2828
- The Cobertura artifact is reused by the CodeScene PR coverage gate and by the develop/manual CodeScene analysis flow.
29-
- If CodeScene flags coverage or duplication, fix the underlying test design instead of suppressing the warning
30-
casually.
29+
- If CodeScene flags coverage or duplication, fix the underlying test design instead of suppressing the warning casually.
3130

3231
## Common pitfalls
3332

0 commit comments

Comments
 (0)