Website updates, RLS spec/plan, build tooling, and reporting cleanup#31
Merged
MelbourneDeveloper merged 4 commits intomainfrom May 1, 2026
Merged
Website updates, RLS spec/plan, build tooling, and reporting cleanup#31MelbourneDeveloper merged 4 commits intomainfrom
MelbourneDeveloper merged 4 commits intomainfrom
Conversation
The reporting.js hand-written vanilla JS React renderer was accidentally deleted in d5f61e1 along with H5-generated build artefacts. The wwwroot/js/ gitignore rule also silently suppressed it. All 13 Playwright browser tests fail because index.html loads js/reporting.js which no longer existed. Switch gitignore from directory-level ignore to explicit per-file ignores so reporting.js (source file) is tracked while H5 build outputs are not.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
TLDR
Adds Row-Level Security spec and implementation plan, a
make vsixtarget for the VS Code extension, website/CSS improvements, and removes committed build artefacts from the Reporting project.What Was Added?
docs/specs/rls-spec.md): 397-line spec covering[RLS-*]sections for PostgreSQL, SQL Server, and SQLite trigger-based emulation, with LQL-transpiled predicates.docs/plans/RLS-PLAN.md): 98-line implementation plan mapping spec sections to Migration framework types (RlsPolicySetDefinition,RlsPolicyDefinition,RlsOperation).make vsixtarget (Makefile): Builds the Rust LSP in release mode, compiles and packages the VS Code extension.vsix, and installs it — wrapsLql/lql-lsp-rust/build-vsix.sh.Website/src/assets/css/styles.css): Added.demo-videoclass with responsive width and border-radius.README.md): Addedlql.pngimage to the Quick Example section.What Was Changed or Deleted?
CodeAnalysis.ruleset: MovedEXHAUSTION001rule to correct position; no functional change.Directory.Build.props: AddedFS3261to performance-critical warnings-as-errors list.DdlGenerator.cs→MigrationRunner.cs: Removed the now-redundantDdlGeneratorstatic helper class (its thinGenerate/GenerateAllwrappers were dead weight); theMigrationRunnertype that shared the file remains.TypeProviderE2ETests.fs): Added explicit type annotations onrowand usedUnchecked.nonNullto satisfyFS3261nullable warnings.Website/package.json: Removed thebuild:sitecomposite script alias.Website/src/docs/samples.md: Significantly trimmed (221 → 42 lines) — condensed description of the Clinical Coding reference implementation, removed redundant detail.wwwroot/js/artefacts: Deleted committed build outputs (Nimblesite.Reporting.React.js,.meta.js,Reporting.React.js,h5.js, vendor bundles,index.html,reporting.js) and added them to.gitignoreso they are no longer tracked.How Do The Automated Tests Prove It Works?
CI was skipped per author request. The F# changes fix compiler warnings (
FS3261) that were already errors underWarningsAsErrors; the TypeProvider E2E tests (TypeProviderE2ETests.fs) would fail to compile without the fix.Spec / Doc Changes
docs/specs/rls-spec.md— new RLS specification with[RLS-*]spec IDs.docs/plans/RLS-PLAN.md— implementation plan referencing those spec IDs.README.mdquick-example section with LQL screenshot.Website/src/docs/samples.md.Breaking Changes