Skip to content

chore(layout): drop store6- prefix from modules - #758

Closed
matt-ramotar wants to merge 3 commits into
MobileNativeFoundation:store6from
matt-ramotar:chore/tidy-store6-layout
Closed

chore(layout): drop store6- prefix from modules#758
matt-ramotar wants to merge 3 commits into
MobileNativeFoundation:store6from
matt-ramotar:chore/tidy-store6-layout

Conversation

@matt-ramotar

Copy link
Copy Markdown
Collaborator

Summary

  • Remove unused Images/ assets and docs/superpowers/plans/.
  • Rename every store6-* Gradle module to an unprefixed directory (store6-core/core/, store6-benchmarks/benchmarks/, and the rest of the set).
  • Update Gradle includes, project accessors, Maven artifact IDs, ABI dump filenames, CI paths, and docs to match. Kotlin packages under org.mobilenativefoundation.store6 are unchanged.

Test plan

  • ./gradlew projects lists the unprefixed modules
  • ./gradlew :core:compileKotlinJvm :core:jvmApiCheck
  • ./gradlew :testing:compileKotlinJvm :testing:jvmApiCheck :compose:compileKotlinJvm :mutations:compileKotlinJvm
  • ./gradlew -p tooling :plugins:test --tests '...SwiftDumpTasksPluginTest'
  • Confirm Store6 CI on this PR uses the new module paths

Made with Cursor

matt-ramotar and others added 3 commits August 16, 2026 17:05
Remove unused Images and superpowers plans, and rename
store6-* Gradle modules to unprefixed directories so the
store6 branch layout matches the published artifact IDs.

Co-authored-by: Cursor <cursoragent@cursor.com>
@matt-ramotar

Copy link
Copy Markdown
Collaborator Author

Opened on the wrong repository; the tidy-up PR targets matt-ramotar/Store6:store6.

@github-project-automation github-project-automation Bot moved this from 🆕 Triage to ✅ Done in Store Roadmap Aug 16, 2026

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes using default effort and found 2 potential issues.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 9726208. Configure here.

llms.txt
compose/README.md
room/README.md
sqldelight/README.md

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Docs sync source list unsorted

High Severity

.github/docs-sync-sources.txt is not in LC_ALL=C sort order: compose/README.md, room/README.md, and sqldelight/README.md appear after llms.txt, but compose/ sorts before docs/. The docs-sync-guard job runs sort -c on every PR and will fail before any docs-change check.

Additional Locations (1)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 9726208. Configure here.

run: |
set -euo pipefail
status=0
for module in extension-probe testing sqldelight compose compose-demo room benchmarks devtools devtools-inspector devtools-demo mutations mutations-quickstart mutations-testing mutations-sqldelight paging-androidx paging-androidx/sample graphql graphql/sample realtime realtime/sample; do

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Internal-access guard skips samples

Medium Severity

The core-internal access sweep includes nested samples for paging-androidx, graphql, and realtime, but omits room/sample and sqldelight/sample even though those directories exist and are built by CI. Internals usage there would not fail the guard.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 9726208. Configure here.

@matt-ramotar
matt-ramotar deleted the chore/tidy-store6-layout branch August 16, 2026 22:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

1 participant