Skip to content

feat(action): publish marketplace metadata at repo root - #42

Merged
mrwogu merged 1 commit into
mainfrom
feat/marketplace-root-action
Sep 21, 2026
Merged

mrwogu merged 1 commit into
mainfrom
feat/marketplace-root-action

Conversation

@mrwogu

@mrwogu mrwogu commented Sep 21, 2026

Copy link
Copy Markdown
Member

Summary

GitHub Marketplace lists only the repository-root action.yml. This adds
a byte-identical root copy of action/action.yml, makes
comarch/git-byline@v1 the primary reference, and keeps the
subdirectory reference working for existing callers. The validate scans
stage now fails when the two copies drift.

Scope

  • User-visible behavior: the composite action is reachable as
    comarch/git-byline@v1; comarch/git-byline/action@v1 is unchanged
  • Interfaces or compatibility: same inputs, same steps, same moving v1
    tag; no breaking change
  • Documentation: README, action/README, SECURITY_MODEL, SUPPLY_CHAIN,
    and RELEASES updated
  • Generated artifacts: none; action/action.yml is hand-maintained, not
    PromptScript output
  • Security and privacy: no runtime behavior change; one new validator
    contract comparing two files inside the repository

Related issue

N/A

Validation

go test ./tools/validate/ -run 'TestCheckActionMetadata|TestCheckScans' -count=1
  ok github.com/comarch/git-byline/tools/validate 0.820s

go run ./tools/validate
  ok gofmt, vet, test, coverage, build, deps, imports, installers,
  promptscript, scans
  validate: all 10 stages passed

Checklist

  • Focused change with unrelated refactors excluded
  • Tests added or updated
  • Documentation updated when behavior changed
  • Generated artifacts regenerated and reviewed (none generated)
  • PromptScript sources updated instead of generated files (none touched)
  • No secrets, private data, or unsafe fixtures included
  • Security impact considered
  • Breaking changes documented (none)
  • Release impact understood (next release needs the marketplace
    checkbox ticked; first publish needs the Developer Agreement and 2FA)

GitHub Marketplace lists only the repository-root action.yml, so the
maintained action/action.yml now ships with a byte-identical root copy
and comarch/git-byline@v1 becomes the primary reference. The validate
scans stage fails when the two files drift, and the subdirectory
reference keeps working for existing callers.

Release procedure documents the per-release marketplace checkbox.
@coderabbitai

coderabbitai Bot commented Sep 21, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 29 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Repository: comarch/git-byline/.coderabbit.yaml

Review profile: ASSERTIVE

Plan: Advanced

Run ID: f8ba15ae-f44d-439b-aa8e-89c6ebf52bfb

📥 Commits

Reviewing files that changed from the base of the PR and between b4f2820 and b2f9955.

📒 Files selected for processing (8)
  • README.md
  • action.yml
  • action/README.md
  • docs/RELEASES.md
  • docs/SECURITY_MODEL.md
  • docs/SUPPLY_CHAIN.md
  • tools/validate/scan.go
  • tools/validate/scan_more_test.go

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@sonarqubecloud

Copy link
Copy Markdown

@codecov

codecov Bot commented Sep 21, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@mrwogu
mrwogu merged commit 0a94a9a into main Sep 21, 2026
19 checks passed
@mrwogu
mrwogu deleted the feat/marketplace-root-action branch September 21, 2026 21:58
@mrwogu mrwogu mentioned this pull request Sep 21, 2026
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.

1 participant