Skip to content

Rename projects to Nimblesite.* namespace, remove Gatekeeper and Samples#27

Merged
MelbourneDeveloper merged 32 commits intomainfrom
breaksamplesout2
Apr 5, 2026
Merged

Rename projects to Nimblesite.* namespace, remove Gatekeeper and Samples#27
MelbourneDeveloper merged 32 commits intomainfrom
breaksamplesout2

Conversation

@MelbourneDeveloper
Copy link
Copy Markdown
Owner

@MelbourneDeveloper MelbourneDeveloper commented Apr 3, 2026

TLDR

Rename all project directories/assemblies to use Nimblesite.* namespace prefix, remove Gatekeeper and Samples (now in separate repos), restructure CI with per-project coverage enforcement, and add comprehensive test coverage.

What Was Added?

  • coverage-thresholds.json — per-project coverage thresholds (88–92% depending on component)
  • GlobalUsings.cs files added to projects that previously lacked them
  • docs/specs/ directory consolidating all specification documents
  • New test files with extensive coverage:
    • CoreCoverageTests.cs (912 lines) — DataProvider core coverage tests
    • GeneratedOperationsCoverageTests.cs (636 lines) — generated operations coverage
    • CodeGenerationExtendedE2ETests.cs (542 lines) — extended code generation E2E tests
    • FunctionMappingE2ETests.cs (293 lines) — LQL function mapping E2E tests
    • LqlDialectE2ETests.cs (462 lines) — LQL dialect E2E tests

What Was Changed or Deleted?

  • Project renames (841 files across the repo): all projects renamed from short names to Nimblesite.* prefix (e.g. DataProviderNimblesite.DataProvider.Core, LqlNimblesite.Lql.Core, SyncNimblesite.Sync.Core, etc.)
  • Deleted Gatekeeper — entire Gatekeeper/ directory removed (WebAuthn + RBAC auth module)
  • Deleted Samples — all healthcare samples removed (moved to separate HealthcareSamples repo)
  • Deleted skillscontainer-logs and run-samples skills removed (referenced deleted Samples)
  • CI workflow restructured — from conditional path-filtered jobs to 3 parallel tracks:
    1. Lint + DataProvider Tests — format check, lint, DataProvider tests with coverage enforcement
    2. Build + LQL / Migration / Sync Tests — full .NET + Rust build, all component tests with per-project coverage thresholds
    3. LQL Extension Tests — TypeScript compile, VS Code extension tests, VSIX packaging
  • CI coverage enforcement — each test project now checked against thresholds in coverage-thresholds.json
  • .NET version set to 9.0.x in CI (was 10.0.x)
  • Removed push-on-main CI trigger — CI now runs only on PRs and manual dispatch
  • Solution file updated — all project references updated to new paths, removed Gatekeeper/Samples projects
  • Spec docs moved — from component directories to docs/specs/
  • TypeScript lint fix — explicit null/undefined check in index.ts to satisfy strict-boolean-expressions

How Do The Automated Tests Prove It Works?

All 1,977 tests pass across every component:

  • DataProvider (462 tests): Nimblesite.DataProvider.Tests (277) + Nimblesite.DataProvider.Example.Tests (185) — source generation, SQLite integration, code generation E2E, core coverage
  • LQL .NET (134 tests): Nimblesite.Lql.Tests — transpiler, dialect E2E, function mapping E2E
  • LQL F# Type Provider (32 tests): Nimblesite.Lql.TypeProvider.FSharp.Tests
  • LQL Rust (485 tests): lql-analyzer (228) + lql-lsp (131) + lsp_protocol (31) + lql-parser (95) — completions, diagnostics, hover, schema, scope, symbols, LSP protocol
  • Migration (283 tests): Nimblesite.DataProvider.Migration.Tests — schema migration framework
  • Sync (581 tests): Core (246) + SQLite (200) + Postgres (54) + Integration (14) + HTTP (67)
  • LQL Extension: TypeScript compiles, VSIX packages successfully

Spec / Doc Changes

  • All spec documents moved from component directories to docs/specs/ (gatekeeper, sync, migration, lql, website, design systems)
  • CLAUDE.md updated to reflect architecture changes
  • Website docs updated to remove Samples references

Breaking Changes

  • All assembly/namespace names changed from short names to Nimblesite.* prefix
  • Gatekeeper removed entirely — consumers must use the separate repo or remove auth dependencies
  • All Samples removed — now live in MelbourneDeveloper/HealthcareSamples
  • CI trigger changed — no longer runs on push to main, only on PRs and manual dispatch

@MelbourneDeveloper MelbourneDeveloper merged commit 8eab0dc into main Apr 5, 2026
1 of 3 checks passed
@MelbourneDeveloper MelbourneDeveloper deleted the breaksamplesout2 branch April 5, 2026 05:29
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