Skip to content

Split community-tools.osl into one file per concern - #71

Merged
Mistium merged 1 commit into
mainfrom
split-community-tools
Nov 1, 2026
Merged

Mistium merged 1 commit into
mainfrom
split-community-tools

Conversation

@Mistium

@Mistium Mistium commented Nov 1, 2026

Copy link
Copy Markdown
Member

Stacked on #69, next to #70 (which splits projects.osl the same way). The two don't touch the same files.

src/community/community-tools.osl was 3,039 lines. It's now:

File Lines Owns
space-model.osl 139 space fields, roles and access checks
space-memberships.osl 158 the membership index and removing projects from spaces
space-responses.osl 153 space JSON for clients
spaces.osl 738 space handlers: create, update, delete, projects, follows, curators, reactions
space-comments.osl 127 space comments
challenges.osl 486 challenge rules, judges, scoring, voting and results
community-user-records.osl 313 purging and renaming a user's community records
releases.osl 218 previews, releases and contributions
playtime-diagnostics.osl 188 playtime sessions and project diagnostics
project-feedback.osl 175 feedback and reviews
roadmap.osl 282 roadmap ideas, votes and comments
share-cards.osl 75 project cards and share images

It's a pure move. All 2,901 non-blank lines are unchanged, checked by comparing the line sets. The file had no top-level statements besides declarations, so initialisation order can't change. Each file's export block lists only the names other directories use.

Checks:

  • osl lsp check main.osl reports no diagnostics, and osl test passes 45/45.
  • osl compile builds.
  • The offline smoke run matches the previous build.

There is no visible change.

@Mistium
Mistium force-pushed the split-community-tools branch from 1d4eba1 to 076dd3a Compare November 1, 2026 07:17
@Mistium
Mistium changed the base branch from src-layout to main November 1, 2026 07:17
community-tools.osl was 3,039 lines covering spaces, their memberships and comments,
challenges, cleanup of a user's community records, releases, playtime and diagnostics,
feedback and reviews, the roadmap, and share cards. Each is now its own file in
src/community. This is a pure move: every non-blank line is unchanged, tests pass and the
offline smoke run matches the previous build.
@Mistium
Mistium force-pushed the split-community-tools branch from 076dd3a to 281dab4 Compare November 1, 2026 07:17
@Mistium
Mistium merged commit 4478bc4 into main Nov 1, 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