Skip to content

Split offstyledb_core.inc into focused include files - #28

Merged
dowoge merged 1 commit into
mainfrom
refactor-file-split
Apr 19, 2026
Merged

Split offstyledb_core.inc into focused include files#28
dowoge merged 1 commit into
mainfrom
refactor-file-split

Conversation

@dowoge

Copy link
Copy Markdown
Member

No behaviour changes. Moves the single ~1460-line include into six smaller ones, each responsible for one area:

  • offstyledb.inc top-level orchestrator: globals, myinfo, AskPluginLoad2, plugin event handlers
  • offstyledb_shavit.inc v3/v4 shavit compat: native/forward stubs, temp replay path helpers, v3 DB stocks
  • offstyledb_http.inc AddHeaders + Crypt_Base64Encode + API_BASE_URL
  • offstyledb_styles.inc style mapping fetch, cache, view/refresh cmds
  • offstyledb_records.inc Shavit_OnFinish/OnReplaySaved, SendRecord and replay upload callbacks
  • offstyledb_bulk.inc bulk verification + SQL scrape + batch pipeline

Prep for the submission-pipeline redesign

No behaviour changes. Moves the single ~1460-line include into six
smaller ones, each responsible for one area:
- offstyledb.inc top-level orchestrator: globals, myinfo, AskPluginLoad2, plugin event handlers
- offstyledb_shavit.inc v3/v4 shavit compat: native/forward stubs, temp replay path helpers, v3 DB stocks
- offstyledb_http.inc AddHeaders + Crypt_Base64Encode + API_BASE_URL
- offstyledb_styles.inc style mapping fetch, cache, view/refresh cmds
- offstyledb_records.inc Shavit_OnFinish/OnReplaySaved, SendRecord and replay upload callbacks
- offstyledb_bulk.inc bulk verification + SQL scrape + batch pipeline
Prep for the submission-pipeline redesign
@dowogedowoge added the enhancement New feature or request label Apr 19, 2026
@dowoge

Copy link
Copy Markdown
MemberAuthor

it compiled so LGTM!!

@dowoge
dowoge merged commit 5207d43 into mainApr 19, 2026
3 checks passed
@dowoge
dowoge deleted the refactor-file-split branch April 19, 2026 13:22
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancementNew feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@dowoge