Skip to content

feat: add draft-segment-schema-pr action - #294

Draft
NicolasMassart wants to merge 2 commits into
mainfrom
feat/mcwp-575-schema-pr-workflow
Draft

NicolasMassart wants to merge 2 commits into
mainfrom
feat/mcwp-575-schema-pr-workflow

Conversation

@NicolasMassart

Copy link
Copy Markdown
Contributor

Summary

  • Adds a reusable draft-segment-schema-pr composite action that reads Mobile/Extension analytics diffs through the GitHub API, comments one proposal, and opens a draft Consensys/segment-schema PR after the author comments I agree to open a draft Segment schema PR.
  • Client file list uses pulls.listFiles (three-dot PR comparison). Schema YAML is still committed and pushed with git. Helper, not a merge gate.
  • Consumer workflows stay in Mobile/Extension; pin a commit SHA until @v1 exists.

Test plan

  • yarn test and yarn lint on this branch
  • Confirm propose posts or edits a single sticky proposal comment and does not comment on PRs with no analytics changes
  • Confirm create opens a draft schema PR only after the author agreement comment on an open, same-repo PR
  • Confirm later propose runs update the same bot branch when a schema PR is already open
  • Confirm close closes the schema PR when the client PR closes unmerged, and leaves it open when merged

Made with Cursor

NicolasMassart and others added 2 commits September 14, 2026 18:46
Open a Consensys/segment-schema draft from Mobile or Extension analytics diffs after the PR author agrees, using the GitHub API for the client file list.

Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@socket-security

socket-security Bot commented Sep 14, 2026

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addedyaml@​2.9.010010010092100

View full report

@socket-security

socket-security Bot commented Sep 14, 2026

Copy link
Copy Markdown

All alerts resolved. Learn more about Socket for GitHub.

This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored.

Ignoring alerts on:

  • yaml@2.9.0

View full report

@NicolasMassart

Copy link
Copy Markdown
Contributor Author

@SocketSecurity ignore npm/yaml@2.9.0

official parser; we only call parseDocument(text), no reviver/onAnchor.

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