feat: add @myorg/todo library with TodoStore (#110)

* feat: add @myorg/todo library with TodoStore (closes #109)

- Todo model: id, title, description, completed, createdAt
- TodoService: getAll, create, update, remove
- TodoStore (scoped): rxResource for fetching, rxMethod for mutations
- enableSync/disableSync toggles API fetching (returns [] when off)
- Todo component: .ts-only, inline template, Tailwind, OnPush

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* feat: build out todo feature with routing and backend API

Frontend:
- TodoList, TodoForm, TodoPage components (.ts-only, Tailwind, OnPush)
- lib.routes.ts with TodoStore scoped to route
- /todos route added to app.routes.ts
- Todos nav link added to shared nav-links
- 16 passing tests across store and components

Backend:
- GET/POST /api/todos, PATCH/DELETE /api/todos/{id}
- In-memory TodoRepository (singleton, ConcurrentDictionary)
- TodoRepository registered in Program.cs
- 8 new unit tests for repository (9 total, all passing)
- Api.http updated with todo request examples

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix: update TodoForm styling to match app form conventions

- External labels (text-xs font-semibold text-on-surface-variant)
- subscriptSizing=dynamic to remove dead space below inputs
- Aligned button to bottom of fields on sm+ screens
- Remove MatLabel import (no longer needed)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix: use mat-label and bg-surface-container in TodoForm

- Restore mat-label inside mat-form-field for proper validation display
- Wrap form in rounded-2xl bg-surface-container card to match other pages
- Keep subscriptSizing=dynamic and sm:flex-row layout

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix: style empty state as alert in TodoList

Replaces plain text with a Shadcn-style alert:
- border border-outline-variant bg-surface-container-low rounded-xl p-4
- inbox icon + title + description

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix: constrain empty state alert to max-w-md

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix: center empty state alert horizontally

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* feat: use signal forms in TodoForm + add curly ESLint rule

- Rewrite TodoForm with @angular/forms/signals (experimental)
  - form() + [formField] directive replaces ReactiveFormsModule
  - required() validator with custom message
  - submit() handles validation + reset
  - computed formValid() drives button disabled state
  - Tailwind-styled native inputs (bg-surface, border-outline, focus:ring-primary)
  - Error shown inline below title input when touched + invalid
- Add curly: ['error', 'all'] to eslint.config.cjs
- Auto-fix existing curly violations in main-toolbar, wait-for-element, todo-form

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix: build error in signal forms + API error handling

Signal forms fix:
- submit() action receives FieldTree not plain model; read values
  via field.title().value() and return Promise.resolve()

Error handling:
- Add mutationError: string | null to TodoState
- Mutation tapResponse.error sets a user-friendly message in state
- Mutation tapResponse.next clears mutationError
- Add clearMutationError() store method
- TodoPage shows dismissible error banner for mutation errors
- TodoPage shows fetch error alert (cloud_off + retry button)
  when store.todos.error() is set, replacing the list

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix: restore mat-label using SignalFormControl from signals/compat

Use SignalFormControl from @angular/forms/signals/compat so the todo
form keeps signal-based validation rules while integrating with
mat-form-field / mat-label / mat-error via standard ReactiveFormsModule
formControlName binding.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix: add curly braces to if statement in update-packages main.ts

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix: enable nullable reference types in Api project

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: chrisjwalk-bot

apps/api/Api.Test/Api.Test.csproj

@@ -21,4 +21,8 @@
     </PackageReference>
   </ItemGroup>
 
+  <ItemGroup>
+    <ProjectReference Include="..\Api\Api.csproj" />
+  </ItemGroup>
+
 </Project>

apps/api/Api.Test/UnitTest.cs

@@ -8,4 +8,94 @@ public void Test()
       Assert.True(true);
     }
   }
+
+  public class TodoRepositoryTests
+  {
+    private static TodoRepository CreateRepo() => new();
+
+    [Fact]
+    public void GetAll_ReturnsEmpty_WhenNoTodosAdded()
+    {
+      var repo = CreateRepo();
+      Assert.Empty(repo.GetAll());
+    }
+
+    [Fact]
+    public void Add_ReturnsTodoWithGeneratedId()
+    {
+      var repo = CreateRepo();
+      var result = repo.Add(new CreateTodoRequest("Buy milk", "From the store", false));
+
+      Assert.NotNull(result.Id);
+      Assert.NotEmpty(result.Id);
+      Assert.Equal("Buy milk", result.Title);
+      Assert.Equal("From the store", result.Description);
+      Assert.False(result.Completed);
+    }
+
+    [Fact]
+    public void Add_MultipleTodos_AreAllReturned()
+    {
+      var repo = CreateRepo();
+      repo.Add(new CreateTodoRequest("First", "", false));
+      repo.Add(new CreateTodoRequest("Second", "", false));
+
+      Assert.Equal(2, repo.GetAll().Count());
+    }
+
+    [Fact]
+    public void Update_ExistingTodo_ReturnsUpdated()
+    {
+      var repo = CreateRepo();
+      var todo = repo.Add(new CreateTodoRequest("Original", "", false));
+
+      var updated = repo.Update(todo.Id, new UpdateTodoRequest("Updated", null, true));
+
+      Assert.NotNull(updated);
+      Assert.Equal("Updated", updated!.Title);
+      Assert.True(updated.Completed);
+      Assert.Equal("", updated.Description); // unchanged
+    }
+
+    [Fact]
+    public void Update_NonExistentId_ReturnsNull()
+    {
+      var repo = CreateRepo();
+      var result = repo.Update("missing-id", new UpdateTodoRequest("x", null, null));
+      Assert.Null(result);
+    }
+
+    [Fact]
+    public void Update_OnlyPatchesProvidedFields()
+    {
+      var repo = CreateRepo();
+      var todo = repo.Add(new CreateTodoRequest("Title", "Desc", false));
+
+      var updated = repo.Update(todo.Id, new UpdateTodoRequest(null, null, true));
+
+      Assert.Equal("Title", updated!.Title);
+      Assert.Equal("Desc", updated.Description);
+      Assert.True(updated.Completed);
+    }
+
+    [Fact]
+    public void Remove_ExistingTodo_ReturnsTrueAndRemoves()
+    {
+      var repo = CreateRepo();
+      var todo = repo.Add(new CreateTodoRequest("Delete me", "", false));
+
+      var removed = repo.Remove(todo.Id);
+
+      Assert.True(removed);
+      Assert.Empty(repo.GetAll());
+    }
+
+    [Fact]
+    public void Remove_NonExistentId_ReturnsFalse()
+    {
+      var repo = CreateRepo();
+      Assert.False(repo.Remove("missing-id"));
+    }
+  }
 }
+

apps/api/Api/Api.csproj

@@ -2,6 +2,7 @@
 
   <PropertyGroup>
     <TargetFramework>net10.0</TargetFramework>
+    <Nullable>enable</Nullable>
   </PropertyGroup>
 
   <ItemGroup>

apps/api/Api/Api.http

@@ -30,3 +30,36 @@ Accept: application/json
 Authorization: Bearer {{access_token}}
 
 ###
+
+### List all todos
+GET {{Api_HostAddress}}/todos
+Accept: application/json
+
+###
+
+### Create a todo
+POST {{Api_HostAddress}}/todos
+Content-Type: application/json
+
+{
+  "title": "Buy groceries",
+  "description": "Milk, eggs, bread",
+  "completed": false
+}
+
+###
+
+### Update a todo (replace <id> with a real ID)
+PATCH {{Api_HostAddress}}/todos/<id>
+Content-Type: application/json
+
+{
+  "completed": true
+}
+
+###
+
+### Delete a todo (replace <id> with a real ID)
+DELETE {{Api_HostAddress}}/todos/<id>
+
+###

apps/api/Api/Controllers/WeatherForecastController.cs

@@ -63,9 +63,9 @@ public string Id
       {
         get { return Guid.NewGuid().ToString(); }
       }
-      public string DateFormatted { get; set; }
+      public string DateFormatted { get; set; } = string.Empty;
       public int TemperatureC { get; set; }
-      public string Summary { get; set; }
+      public string Summary { get; set; } = string.Empty;
 
       public int TemperatureF
       {

apps/api/Api/Endpoints/TodoEndpoints.cs

@@ -0,0 +1,85 @@
+using System;
+using System.Collections.Concurrent;
+using System.Collections.Generic;
+using System.Linq;
+using Microsoft.AspNetCore.Builder;
+using Microsoft.AspNetCore.Http;
+using Microsoft.AspNetCore.Routing;
+
+public record TodoItem(
+  string Id,
+  string Title,
+  string Description,
+  bool Completed,
+  string CreatedAt
+);
+
+public record CreateTodoRequest(string Title, string Description, bool Completed = false);
+
+public record UpdateTodoRequest(string? Title, string? Description, bool? Completed);
+
+public class TodoRepository
+{
+  private readonly ConcurrentDictionary<string, TodoItem> _todos = new();
+
+  public IEnumerable<TodoItem> GetAll() =>
+    _todos.Values.OrderBy(t => t.CreatedAt);
+
+  public TodoItem Add(CreateTodoRequest req)
+  {
+    var todo = new TodoItem(
+      Id: Guid.NewGuid().ToString(),
+      Title: req.Title,
+      Description: req.Description,
+      Completed: req.Completed,
+      CreatedAt: DateTime.UtcNow.ToString("O")
+    );
+    _todos[todo.Id] = todo;
+    return todo;
+  }
+
+  public TodoItem? Update(string id, UpdateTodoRequest req)
+  {
+    if (!_todos.TryGetValue(id, out var existing))
+      return null;
+
+    var updated = existing with
+    {
+      Title = req.Title ?? existing.Title,
+      Description = req.Description ?? existing.Description,
+      Completed = req.Completed ?? existing.Completed,
+    };
+    _todos[id] = updated;
+    return updated;
+  }
+
+  public bool Remove(string id) => _todos.TryRemove(id, out _);
+}
+
+public static class TodoEndpoints
+{
+  public static IEndpointRouteBuilder MapTodoEndpoints(this IEndpointRouteBuilder app)
+  {
+    var group = app.MapGroup("/api/todos").WithTags("Todos");
+
+    group.MapGet("", (TodoRepository repo) =>
+      Results.Ok(repo.GetAll()));
+
+    group.MapPost("", (CreateTodoRequest req, TodoRepository repo) =>
+    {
+      var todo = repo.Add(req);
+      return Results.Created($"/api/todos/{todo.Id}", todo);
+    });
+
+    group.MapPatch("{id}", (string id, UpdateTodoRequest req, TodoRepository repo) =>
+    {
+      var todo = repo.Update(id, req);
+      return todo is null ? Results.NotFound() : Results.Ok(todo);
+    });
+
+    group.MapDelete("{id}", (string id, TodoRepository repo) =>
+      repo.Remove(id) ? Results.NoContent() : Results.NotFound());
+
+    return app;
+  }
+}

apps/api/Api/Program.cs

@@ -83,6 +83,7 @@
   .AddApiEndpoints();
 
 builder.Services.AddScoped<TokenService>();
+builder.Services.AddSingleton<TodoRepository>();
 
 var connectionString = builder.Environment.IsDevelopment()
   ? builder.Configuration.GetConnectionString("AZURE_SQL_CONNECTIONSTRING")
@@ -197,6 +198,9 @@
 // Custom JWT auth endpoints: login / refresh / logout
 app.MapAuthEndpoints(app.Environment.IsDevelopment());
 
+// Todo CRUD endpoints (in-memory store)
+app.MapTodoEndpoints();
+
 // Keep Identity account-management endpoints (password reset, email confirmation, 2FA setup, etc.)
 // Login and refresh from this group are superseded by /api/auth/* above.
 app.MapGroup("/api/account")

apps/web-app/src/app/app.routes.ts

@@ -14,6 +14,10 @@ export const routes: Route[] = [
     loadChildren: () =>
       import('@myorg/weather-forecast').then((m) => m.weatherForecastRoutes),
   },
+  {
+    path: 'todos',
+    loadChildren: () => import('@myorg/todo').then((m) => m.todoRoutes),
+  },
   {
     path: 'login',
     loadChildren: () => import('@myorg/login').then((m) => m.loginRoutes),

eslint.config.cjs

@@ -33,10 +33,10 @@ module.exports = [
     files: ['**/*.ts', '**/*.tsx', '**/*.js', '**/*.jsx'],
     // Override or add rules here
     rules: {
-
       '@/semi': ['error', 'always'],
       '@/no-extra-semi': 'error',
       '@/quotes': ['error', 'single', { allowTemplateLiterals: true }],
+      curly: ['error', 'all'],
       '@angular-eslint/component-class-suffix': 'off',
     },
   },

libs/shared/src/lib/components/main-toolbar.ts

@@ -164,15 +164,23 @@ export class MainToolbar {
 
   readonly themeIcon = computed(() => {
     const t = this.themeService.theme();
-    if (t === 'light') return 'light_mode';
-    if (t === 'dark') return 'dark_mode';
+    if (t === 'light') {
+      return 'light_mode';
+    }
+    if (t === 'dark') {
+      return 'dark_mode';
+    }
     return 'brightness_auto';
   });
 
   readonly themeTooltip = computed(() => {
     const t = this.themeService.theme();
-    if (t === 'light') return 'Theme: Light (click for Dark)';
-    if (t === 'dark') return 'Theme: Dark (click for System)';
+    if (t === 'light') {
+      return 'Theme: Light (click for Dark)';
+    }
+    if (t === 'dark') {
+      return 'Theme: Dark (click for System)';
+    }
     return 'Theme: System (click for Light)';
   });
 }