Skip to content

Publish a Compose field support matrix - #30

Merged
GaTTGeng merged 11 commits into
masterfrom
issue-13-field-matrix
Aug 3, 2026
Merged

Publish a Compose field support matrix#30
GaTTGeng merged 11 commits into
masterfrom
issue-13-field-matrix

Conversation

@GaTTGeng

@GaTTGeng GaTTGeng commented Aug 3, 2026

Copy link
Copy Markdown
Owner

Closes #13.

What changed

  • Add a maintained matrix for every field exposed by ServiceDefinition and ComposeProject.
  • Distinguish applied, partial, parsed-only, unsupported, and planned behavior.
  • Link related implementation gaps to their tracked issues.
  • Add matrix links from the English and Simplified Chinese READMEs and the roadmap.

Why

The loader retains more Compose-shaped data than the engine currently applies. The matrix makes that boundary visible before users rely on a property at runtime.

Impact

Consumers can quickly verify whether a field is applied, only retained for inspection, or part of tracked future work.

Validation

  • Verified all 62 ServiceDefinition members appear in the matrix.
  • git diff --check
  • dotnet pack ComposeSharp.sln --configuration Release --no-build --no-restore --output artifacts/packages
  • ./scripts/VerifyPackageReadme.ps1

@GaTTGeng
GaTTGeng marked this pull request as ready for review August 3, 2026 01:37
Copilot AI review requested due to automatic review settings August 3, 2026 01: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 adds a maintained “parsed vs applied” support matrix for ComposeSharp’s exposed Compose model, making it clear which ServiceDefinition and ComposeProject fields affect Docker Engine behavior versus being retained only for inspection.

Changes:

  • Add docs/compose-field-matrix.md documenting per-field status (applied/partial/parsed-only/unsupported/planned) and linking to tracked gaps.
  • Link to the matrix from both READMEs (English + zh-CN) to set correct runtime expectations.
  • Update the roadmap to point to the matrix as the canonical “parsed vs applied” documentation.

Reviewed changes

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

File Description
README.md Adds user-facing guidance and a roadmap section link to the new support matrix.
README.zh-CN.md Adds the same guidance/links in Simplified Chinese, aligning expectations across locales.
docs/roadmap.md References the matrix from milestone 2.1 exit criteria to anchor “parsed vs applied” documentation.
docs/compose-field-matrix.md New documentation file defining statuses and listing service + top-level project fields with current behavior.

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

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: a3e55fc2bc

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread docs/compose-field-matrix.md Outdated
Comment thread docs/compose-field-matrix.md Outdated
@GaTTGeng
GaTTGeng force-pushed the issue-13-field-matrix branch from cc7c0bd to 210ea11 Compare August 3, 2026 01:44

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 210ea11876

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread docs/compose-field-matrix.md Outdated
Comment thread docs/compose-field-matrix.md Outdated
Comment thread docs/compose-field-matrix.md Outdated

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: abadee0081

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread docs/compose-field-matrix.md Outdated
Comment thread docs/compose-field-matrix.md Outdated
Comment thread docs/compose-field-matrix.md Outdated
Comment thread docs/compose-field-matrix.md Outdated
Comment thread docs/compose-field-matrix.md Outdated

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 8226ee57ec

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread docs/compose-field-matrix.md Outdated
Comment thread docs/compose-field-matrix.md Outdated
Comment thread docs/compose-field-matrix.md Outdated

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 513ccb7d39

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread docs/compose-field-matrix.md Outdated
Comment thread docs/compose-field-matrix.md Outdated

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: c1f198f790

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread docs/compose-field-matrix.md Outdated
Comment thread docs/compose-field-matrix.md Outdated
Comment thread docs/compose-field-matrix.md Outdated

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 71ac6bbf60

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread docs/compose-field-matrix.md Outdated
Comment thread docs/compose-field-matrix.md Outdated
Comment thread docs/compose-field-matrix.md Outdated

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 5ecf84460b

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread docs/compose-field-matrix.md Outdated
Comment thread docs/compose-field-matrix.md Outdated

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 8758768c16

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread docs/compose-field-matrix.md Outdated
Comment thread docs/compose-field-matrix.md Outdated
Comment thread docs/compose-field-matrix.md Outdated
Comment thread docs/compose-field-matrix.md Outdated

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 24b995c941

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread docs/compose-field-matrix.md Outdated
@GaTTGeng
GaTTGeng merged commit a6d913c into master Aug 3, 2026
1 check passed
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.

Publish a parsed-versus-applied Compose field matrix

2 participants