Skip to content

chore: Sync generated JSON schemas (adds personas to insights) - #19

Merged
cristidrg merged 2 commits into
cristidrg/docs-import-field-semanticsfrom
cristidrg/sync-schemas-pr-26312
Sep 10, 2026
Merged

chore: Sync generated JSON schemas (adds personas to insights)#19
cristidrg merged 2 commits into
cristidrg/docs-import-field-semanticsfrom
cristidrg/sync-schemas-pr-26312

Conversation

@cristidrg

@cristidrg cristidrg commented Sep 10, 2026

Copy link
Copy Markdown

Description

Refreshes all six schema files from the monorepo generator (yarn workspace @theydo/external-format refresh).

Substantive change — only one:

  • InsightsFile.schema.json: adds the personas field (array of persona share URLs …/p/<shareKey>, nullable). Personas that do not exist in the target workspace are ignored — imports never create personas.

Everything else is generator churn, verified semantically equivalent:

  • SurveyResponsesFile / FeedbackResponsesFile: $ref/$defs indirection inlined; resolved schemas are byte-identical to the old ones.
  • MetricsFile, OpportunitiesFile, SolutionsFile: key ordering/formatting only — zero semantic changes.

Companion to the README update in #18, kept separate as requested.

Not included: an examples/insights.json (the repo has never had one; can follow up if wanted).

🤖 Generated with Claude Code

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@cristidrg
cristidrg marked this pull request as ready for review September 10, 2026 12:02
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@cristidrg

Copy link
Copy Markdown
Author

Added a follow-up commit: the ownerEmail description in the four entity schemas now states the matching rule inline (matched to existing workspace users, never created, unresolvable email leaves the owner unchanged) instead of pointing to the README section this repo is replacing. Source change: theydo/theydo#26368.

@cristidrg
cristidrg changed the base branch from main to cristidrg/docs-import-field-semantics September 10, 2026 12:19
@cristidrg
cristidrg merged commit e6c83c5 into cristidrg/docs-import-field-semantics Sep 10, 2026
2 checks 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.

1 participant