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

@fqjonyfqjony 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
fqjonyforce-pushed the fix/post-0.20-copilot-followups branch 3 times, most recently from c7a5ae0 to 235e407CompareJuly 10, 2026 09:35
@fqjony
fqjony marked this pull request as ready for review July 10, 2026 09:37
CopilotAI review requested due to automatic review settings July 10, 2026 09:37

CopilotAI 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
FileDescription
tests/suite.shAdds regression coverage for excluding gitignored root YAML manifests; validates .rabbit/README.md scaffolding behavior.
lib/modules/repo_scaffold.shSeeds .rabbit/README.md if missing; changes declared root YAML discovery to use repo file filtering.
package.jsonBumps package version to 0.20.1.
package-lock.jsonSyncs lockfile version metadata to 0.20.1.
changes.mdAdds 0.20.1 release notes describing the follow-ups.
.rabbit/README.mdAdds a Rabbit-facing README stub for this repo.
.rabbit/context.yamlUpdates generated context metadata to 0.20.1.

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

Comment threadlib/modules/repo_scaffold.sh Outdated
Comment threadchanges.md
CopilotAI review requested due to automatic review settings July 10, 2026 09:41
@fqjony
fqjonyforce-pushed the fix/post-0.20-copilot-followups branch from 235e407 to eb332f7CompareJuly 10, 2026 09:41

CopilotAI 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 threadtests/suite.sh Outdated
Comment threadchanges.md
@fqjony
fqjony requested review from a team and udx-githubJuly 10, 2026 09:49
@fqjonyfqjony self-assigned this Jul 10, 2026
CopilotAI review requested due to automatic review settings July 10, 2026 09:50
@fqjony
fqjonyforce-pushed the fix/post-0.20-copilot-followups branch from eb332f7 to c7a33eeCompareJuly 10, 2026 09:50
@fqjonyfqjony changed the title Prepare post 0.20.0 follow-upsPrepare dev.kit 0.20.1 context scaffold fixesJul 10, 2026

CopilotAI 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 latestJul 10, 2026
10 checks passed
@fqjony
fqjony deleted the fix/post-0.20-copilot-followups branch July 10, 2026 09:56
Sign up for freeto 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

@fqjony@udx-github