Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions .github/WORKFLOWS.md
Original file line number Diff line number Diff line change
Expand Up @@ -151,10 +151,10 @@ Enable these settings in your repository:
Add these badges to your README:

```markdown
[![CI](https://github.com/idotta/jsonb-store/actions/workflows/ci.yml/badge.svg)](https://github.com/idotta/jsonb-store/actions/workflows/ci.yml)
[![Code Quality](https://github.com/idotta/jsonb-store/actions/workflows/code-quality.yml/badge.svg)](https://github.com/idotta/jsonb-store/actions/workflows/code-quality.yml)
[![CI](https://github.com/idotta/lite-doc-store/actions/workflows/ci.yml/badge.svg)](https://github.com/idotta/lite-doc-store/actions/workflows/ci.yml)
[![Code Quality](https://github.com/idotta/lite-doc-store/actions/workflows/code-quality.yml/badge.svg)](https://github.com/idotta/lite-doc-store/actions/workflows/code-quality.yml)
[![NuGet](https://img.shields.io/nuget/v/LiteDocumentStore.svg)](https://www.nuget.org/packages/LiteDocumentStore/)
[![codecov](https://codecov.io/gh/idotta/jsonb-store/branch/main/graph/badge.svg)](https://codecov.io/gh/idotta/jsonb-store)
[![codecov](https://codecov.io/gh/idotta/lite-doc-store/branch/main/graph/badge.svg)](https://codecov.io/gh/idotta/lite-doc-store)
```

## Troubleshooting
Expand Down
7 changes: 4 additions & 3 deletions .github/instructions/code-style.instructions.md
Original file line number Diff line number Diff line change
Expand Up @@ -86,9 +86,10 @@ SELECT * FROM [Customer] WHERE json_extract(data, '$.email') = @Email
### Parameterization

```csharp
// ALWAYS use parameters, never string concatenation
var sql = "SELECT * FROM [Customer] WHERE id = @Id";
await _connection.QueryAsync(sql, new { Id = id });
// ALWAYS use parameters, never string concatenation.
// Use the raw ADO.NET helpers in Core/SqliteCommandExtensions.cs (bind by name):
var sql = "SELECT json(data) FROM [Customer] WHERE id = @Id";
var json = await _connection.QueryFirstStringAsync(sql, ("Id", id));

// NEVER do this
var sql = $"SELECT * FROM [Customer] WHERE id = '{id}'"; // SQL INJECTION!
Expand Down
14 changes: 7 additions & 7 deletions .github/instructions/general.instructions.md
Original file line number Diff line number Diff line change
Expand Up @@ -12,9 +12,8 @@ The goal is NOT an opaque document database. Users should seamlessly mix documen

- **.NET 10** - Target framework
- **SQLite 3.45+** - Required for JSONB support
- **Microsoft.Data.Sqlite** - SQLite provider
- **Dapper** - Micro-ORM for minimal mapping overhead
- **System.Text.Json** - Fixed, optimized JSON serializer
- **Microsoft.Data.Sqlite** - SQLite provider (raw ADO.NET; no ORM)
- **System.Text.Json** - AOT-safe serializer via `JsonTypeInfo<T>`

## Key Architectural Principles

Expand All @@ -30,9 +29,9 @@ The goal is NOT an opaque document database. Users should seamlessly mix documen
```
src/
├── LiteDocumentStore/ # Main library
│ ├── DocumentStore.cs # Core repository implementation
│ ├── SqliteJsonbTypeHandler.cs # Dapper type handler for JSONB
│ └── JsonHelper.cs # Optimized JSON serialization
│ ├── Core/DocumentStore.cs # Core store implementation
│ ├── Core/SqliteCommandExtensions.cs # Raw ADO.NET helpers (replaced Dapper)
│ └── Serialization/JsonHelper.cs # AOT-safe JSON serialization
└── tests/
├── LiteDocumentStore.UnitTests/ # Unit tests (mocked)
└── LiteDocumentStore.IntegrationTests/ # Integration tests (real SQLite)
Expand All @@ -41,7 +40,8 @@ src/

## When Modifying This Project

- Check `IMPLEMENTATION_CHECKLIST.md` for the roadmap
- See `CLAUDE.md` at the repo root for the authoritative architecture guide
- Keep the library AOT-clean: no reflection-based serialization, no `Expression.Compile`, no `dynamic`
- All public APIs need XML documentation
- New features need both unit and integration tests
- Consider performance implications of every change
Expand Down
227 changes: 0 additions & 227 deletions .github/instructions/patterns.instructions.md

This file was deleted.

Loading