Skip to content

Prepare dev.kit 0.20.1 context scaffold fixes#30

Merged
fqjony merged 1 commit into
latestfrom
fix/post-0.20-copilot-followups
Jul 10, 2026
Merged

Prepare dev.kit 0.20.1 context scaffold fixes#30
fqjony merged 1 commit into
latestfrom
fix/post-0.20-copilot-followups

Conversation

@fqjony

@fqjony fqjony commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Summary

  • exclude gitignored root YAML files from declared contract manifest discovery
  • seed missing .rabbit/README.md files so new repos have a Rabbit-facing notes surface without adding a generic docs/README.md stub
  • reserve 0.20.1 for the next patch release

Context

These follow up on post-merge Copilot comments on PR #29 plus the worker-php follow-up that moved Rabbit CI notes under .rabbit/README.md. They are not critical for the already published 0.20.0 release, but should be included in the next patch release.

Validation

  • bash tests/suite.sh --only repo-contract
  • bash tests/suite.sh --only core
  • make test
  • git diff --check

@fqjony fqjony force-pushed the fix/post-0.20-copilot-followups branch 3 times, most recently from c7a5ae0 to 235e407 Compare July 10, 2026 09:35
@fqjony fqjony marked this pull request as ready for review July 10, 2026 09:37
Copilot AI review requested due to automatic review settings July 10, 2026 09:37

Copilot AI left a comment

Copy link
Copy Markdown

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 prepares the repo for the post-0.20.0 patch release by tightening declared root YAML contract manifest discovery to respect gitignore-aligned file filtering, and by improving the default repo scaffold to include a Rabbit-facing README in .rabbit/.

Changes:

  • Update root YAML manifest discovery to use repo “content files” filtering (so gitignored root YAML isn’t promoted into generated context).
  • Add scaffolding for .rabbit/README.md (and add/adjust tests to ensure it’s created only when missing and never overwrites an existing one).
  • Bump version metadata to 0.20.1 and document the release in changes.md.

Reviewed changes

Copilot reviewed 6 out of 7 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
tests/suite.sh Adds regression coverage for excluding gitignored root YAML manifests; validates .rabbit/README.md scaffolding behavior.
lib/modules/repo_scaffold.sh Seeds .rabbit/README.md if missing; changes declared root YAML discovery to use repo file filtering.
package.json Bumps package version to 0.20.1.
package-lock.json Syncs lockfile version metadata to 0.20.1.
changes.md Adds 0.20.1 release notes describing the follow-ups.
.rabbit/README.md Adds a Rabbit-facing README stub for this repo.
.rabbit/context.yaml Updates generated context metadata to 0.20.1.

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

Comment thread lib/modules/repo_scaffold.sh Outdated
Comment thread changes.md
Copilot AI review requested due to automatic review settings July 10, 2026 09:41
@fqjony fqjony force-pushed the fix/post-0.20-copilot-followups branch from 235e407 to eb332f7 Compare July 10, 2026 09:41

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 8 out of 9 changed files in this pull request and generated 2 comments.

Comment thread tests/suite.sh Outdated
Comment thread changes.md
@fqjony fqjony requested review from a team and udx-github July 10, 2026 09:49
@fqjony fqjony self-assigned this Jul 10, 2026
Copilot AI review requested due to automatic review settings July 10, 2026 09:50
@fqjony fqjony force-pushed the fix/post-0.20-copilot-followups branch from eb332f7 to c7a33ee Compare July 10, 2026 09:50
@fqjony fqjony changed the title Prepare post 0.20.0 follow-ups Prepare dev.kit 0.20.1 context scaffold fixes Jul 10, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 8 out of 9 changed files in this pull request and generated no new comments.

@fqjony fqjony merged commit 586783d into latest Jul 10, 2026
10 checks passed
@fqjony fqjony deleted the fix/post-0.20-copilot-followups branch July 10, 2026 09:56
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