Skip to content

fix: preserve media metadata in SQL exports - #49

Open
ShiroKSH wants to merge 1 commit into
hasaneyldrm:mainfrom
ShiroKSH:fix/preserve-media-metadata
Open

fix: preserve media metadata in SQL exports#49
ShiroKSH wants to merge 1 commit into
hasaneyldrm:mainfrom
ShiroKSH:fix/preserve-media-metadata

Conversation

@ShiroKSH

Copy link
Copy Markdown

Summary

  • Preserve media_id and required Gym visual attribution in every generated SQL schema and INSERT statement.
  • Apply the fix to both shipped SQL exporters: the standalone setup guide and the embedded browser exporter.
  • Include both fields in the generated backend prompt's dataset contract.

Root cause

The exporters created and populated only 20 columns despite each dataset record carrying media_id and attribution. Generated imports silently discarded the original media reference and required credit.

Checks

  • Generated and inspected DDL/INSERT output for MSSQL, PostgreSQL, MySQL, and SQLite in both exporters.
  • Imported all 1,324 records into SQLite from each exporter; media_id and attribution were populated for every row.

@ShiroKSH

Copy link
Copy Markdown
Author

@hasaneyldrm We'd appreciate it if you could take a look at this pull request when you have a chance.

@ShiroKSH
ShiroKSH force-pushed the fix/preserve-media-metadata branch from 1ba68c0 to 0e19630 Compare August 8, 2026 10:50
@ShiroKSH

ShiroKSH commented Aug 8, 2026

Copy link
Copy Markdown
Author

Hi maintainers, this PR has been open for over two weeks. When you have a moment, I would appreciate a review. Thank you.

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