Skip to content

V0.11.2/service update - #47

Merged
gimlichael merged 21 commits into
mainfrom
v0.11.2/service-update
Oct 3, 2026
Merged

gimlichael merged 21 commits into
mainfrom
v0.11.2/service-update

Conversation

@gimlichael

Copy link
Copy Markdown
Member

This pull request enables git-remote-pr to publish clean feature branches without upstream tracking and expands the release-note workflows for versioned changelogs and complete NuGet package coverage. It also updates repository guidance and aligns deterministic validation specifications with these behaviors.

First-publication PR workflow:

  • Plans publication to origin or the sole GitHub remote using the current branch name, then establishes upstream tracking as part of the approved push.
  • Verifies the published branch and tracking before writing PR metadata, while rejecting ambiguous remotes and incomplete or mismatched tracking.

Versioned changelog behavior:

  • Gives yolo and auto the same concrete-version behavior, removes [Unreleased] only when its contents can be reconciled, and stops safely when the version or existing content is unresolved.
  • Preserves ordinary [Unreleased] behavior and adds regression specifications for both autonomy modes and their boundaries.

Complete NuGet release coverage:

  • Resolves the full branch delta against the integration branch by default, independent of same-named tracking branches, while preserving explicit range selection.
  • Requires a complete version and availability block for every selected packable project, including unchanged packages with the defined default ALM entry.

Repository release guidance:

  • Records the 0.11.2 release outcome and keeps the repository skill catalog aligned with the updated release-note workflows.

Contract validation:

  • Extends deterministic checks to cover the updated skill contracts and package coverage requirements.

Explain how untracked branches are published and how upstream tracking is established during PR execution.
Allow a clean branch without tracking to prepare an explicit first-publication plan, while preserving fail-closed handling for ambiguous remotes and mismatched tracking.
Let the approved first-publication plan push an untracked branch and establish its upstream, with deterministic coverage for remote selection, plan binding, and post-push checks.
Record the git-remote-pr release outcome and its version comparison link.
Keep the repository skill catalog aligned with the updated changelog and NuGet release-note workflows.
Define concrete-version handling and safe removal of Unreleased content for yolo and auto requests.
Use the integration-branch merge-base by default and regenerate existing version blocks from the full branch delta.
Keep deterministic repository assertions aligned with both release-note skill contracts.
Require complete versioned notes for every selected packable project, including packages with no surviving semantic changes.
Assert the package coverage and default ALM requirements in deterministic skill validation.
Describe complete package coverage for version-only release-note requests in the repository skill catalog.
@gimlichael gimlichael self-assigned this Oct 3, 2026
@greptile-apps

greptile-apps Bot commented Oct 3, 2026 •

Copy link
Copy Markdown

RetriggerConfidence Score: 5/5

[Medium risk] Updates skill documentation, test suites, and validation workflows.

The PR appears safe to merge on the new changes reviewed, although the existing lone-flag validation finding remains outstanding.

Fix All in CodexFindings

  1. P2 Incomplete flag passes validation ▶
Fix with agent prompt
### Issue 1
skills/git-visual-commits/scripts/validate-commit-subject.ps1:54-55
A subject such as `🇩 update budget` passes because the lone regional indicator falls within the bundled emoji range and counts as one grapheme. But it is not a complete flag emoji. This lets an invalid emoji-first subject through the validator.

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

Summary

The PR enables first-time PR publication from branches without upstream tracking, revises changelog and NuGet release-note guidance, broadens emoji selection, and separates PR workflow validation into its own suite.

  • Publication plans now bind upstream setup to the approved push and verify tracking before PR metadata writes.
  • Release-note guidance covers versioned changelogs and all selected packable projects, including unchanged ones.
  • Deterministic tests and repository guidance reflect the revised workflows.

Reviews (5) · Last reviewed commit: "✅ stabilize progress coalescing regressi..."

Comment thread skills/git-nuget-release-notes/SKILL.md
Treat the bundled emoji tables as guidance so user choices and repository conventions can express intent beyond the listed examples while preserving subject formatting rules.
Remove the curated-table membership restriction so a single Unicode symbol sequence can represent the change, including flags, modifiers, ZWJ sequences, and keycaps.
Protect the broader Unicode selection contract with positive cases for extended emoji sequences and negative cases for malformed subjects, while aligning repository assertions with the revised guidance.
Describe the release scope across PR publication, changelog handling, NuGet release-note coverage, and Unicode emoji selection so readers can see the complete workflow changes.
Comment thread skills/git-visual-commits/scripts/validate-commit-subject.ps1 Outdated
Comment thread skills/git-visual-commits/scripts/validate-commit-subject.ps1 Outdated
The validator previously accepted ordinary Unicode symbols as emoji. Unicode Emoji property data makes accepted subjects follow the emoji contract consistently.
Document the offline Unicode property data used by the validator so maintainers can keep its accepted emoji set current.
Comment on lines +54 to +55
$isSingleEmoji = [System.Globalization.StringInfo]::ParseCombiningCharacters($emoji).Count -eq 1 -and
($hasEmojiBase -or $emoji -match '^[0-9#*]\uFE0F?\u20E3$')

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Incomplete flag passes validation A subject such as 🇩 update budget passes because the lone regional indicator falls within the bundled emoji range and counts as one grapheme. But it is not a complete flag emoji. This lets an invalid emoji-first subject through the validator.

Prompt To Fix With AI
This is a comment left during a code review.
Path: skills/git-visual-commits/scripts/validate-commit-subject.ps1
Line: 54-55

Comment:
**Incomplete flag passes validation** A subject such as `🇩 update budget` passes because the lone regional indicator falls within the bundled emoji range and counts as one grapheme. But it is not a complete flag emoji. This lets an invalid emoji-first subject through the validator.

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

Fix in Codex

Separating PR routing regressions keeps the template suite focused and lets CI and local runs schedule that work independently.
Running the PR regression suite as a separate CI job lets it share the matrix while preserving the existing validation check.
Documenting the added suite keeps the local validation guidance aligned with the expanded CI matrix.
The timing assertion treated delayed relay reads as a coalescing failure. Keep subprocess coverage for relay delivery and exercise heartbeat suppression with controlled timestamps.
@gimlichael
gimlichael merged commit eec1bda into main Oct 3, 2026
15 checks passed
@gimlichael
gimlichael deleted the v0.11.2/service-update branch October 3, 2026 15:57
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.

2 participants