Merge origin/master into shub/table-name-to-accessor

Author: clockwork-labs-bot

.github/GREMLINS.md

@@ -0,0 +1,33 @@
+# GREMLINS.md — Who Lives in the Pipes
+
+This file documents the automated agents and bots that operate on this repository.
+
+## Clawd 🔧
+
+- **What:** Anti-entropy gremlin / CI watchdog
+- **GitHub account:** `clockwork-labs-bot`
+- **Discord channel:** #gremlins (CL - SpacetimeDB)
+- **Powered by:** [OpenClaw](https://github.com/openclaw/openclaw) + Claude
+
+### What Clawd does
+
+- **Monitors CI** — watches for failures, flaky tests, and regressions
+- **Reviews PRs** — comments on obvious bugs (never approves)
+- **Surfaces stale PRs** — finds approved PRs that just need a rebase
+- **Documents testing** — creates and maintains `DEVELOP.md` files explaining CI and test infrastructure
+- **Alerts the team** — posts findings in #gremlins, pings DevOps when something needs attention
+
+### What Clawd does NOT do
+
+- Approve or merge PRs
+- Take any destructive action (delete branches, close PRs, force push)
+- Modify production infrastructure
+
+### Contacting Clawd
+
+- Mention `@Openclaw` in #gremlins on Discord
+- Tag `@clockwork-labs-bot` on GitHub PRs/issues
+
+---
+
+*To add a new gremlin, document it here.*

.github/workflows/ci.yml

@@ -164,6 +164,13 @@ jobs:
           }
           cargo ci smoketests -- --test-threads=1          
 
+      - name: Check for changes
+        run: |
+          tools/check-diff.sh crates/smoketests || {
+              echo 'Error: There is a diff in the smoketests directory.'
+              exit 1
+          }
+
   smoketests-python:
     needs: [lints]
     name: Smoketests (Python Legacy) (${{matrix.name}})

crates/bindings-csharp/Runtime/Internal/Autogen/ExplicitNameEntry.g.cs

@@ -150,7 +150,9 @@ export class ModuleContext {
     views: [],
     lifeCycleReducers: [],
     caseConversionPolicy: { tag: 'SnakeCase' },
-    explicitNames: { entries: [] },
+    explicitNames: {
+      entries: [],
+    },
   };
 
   get moduleDef(): ModuleDef {

crates/bindings-typescript/src/lib/autogen/explicit_name_entry_type.ts

@@ -84,12 +84,13 @@ pub enum RawModuleDefV10Section {
 
     RowLevelSecurity(Vec<RawRowLevelSecurityDefV10>), //TODO: Add section for Event tables, and Case conversion before exposing this from module
 
+    /// Case conversion policy for identifiers in this module.
     CaseConversionPolicy(CaseConversionPolicy),
 
+    /// Names provided explicitly by the user that do not follow from the case conversion policy.
     ExplicitNames(ExplicitNames),
 }
 
-/// Specifies how identifiers should be converted when interpreting module definitions.
 #[derive(Debug, Clone, Copy, Default, SpacetimeType)]
 #[cfg_attr(feature = "test", derive(PartialEq, Eq, PartialOrd, Ord))]
 #[sats(crate = crate)]
@@ -111,7 +112,26 @@ pub enum CaseConversionPolicy {
 #[cfg_attr(feature = "test", derive(PartialEq, Eq, Ord, PartialOrd))]
 #[non_exhaustive]
 pub struct NameMapping {
+    /// The original name as defined or generated inside module.
+    ///
+    /// Generated as:
+    /// - Tables: value from `#[spacetimedb::table(accessor = ...)]`.
+    /// - Reducers/Procedures/Views: function name
+    /// - Indexes: `{table_name}_{column_names}_idx_{algorithm}`
+    ///
+    /// During validation, this may be replaced by `canonical_name`
+    /// if an explicit or policy-based name is applied.
     pub source_name: RawIdentifier,
+
+    /// The canonical identifier used in system tables and client code generation.
+    ///
+    /// Set via:
+    /// - `#[spacetimedb::table(name = "...")]` for tables
+    /// - `#[spacetimedb::reducer(name = "...")]` for reducers
+    /// - `#[name("...")]` for other entities
+    ///
+    /// If not explicitly provided, this defaults to `source_name`
+    /// after validation. No particular format should be assumed.
     pub canonical_name: RawIdentifier,
 }
 
@@ -122,13 +142,18 @@ pub struct NameMapping {
 pub enum ExplicitNameEntry {
     Table(NameMapping),
     Function(NameMapping),
+    Index(NameMapping),
 }
 
 #[derive(Debug, Default, Clone, SpacetimeType)]
 #[sats(crate = crate)]
 #[cfg_attr(feature = "test", derive(PartialEq, Eq, Ord, PartialOrd))]
 #[non_exhaustive]
 pub struct ExplicitNames {
+    /// Explicit name mappings defined in the module.
+    ///
+    /// These override policy-based or auto-generated names
+    /// during schema validation.
     entries: Vec<ExplicitNameEntry>,
 }