chore(deps): bump DavidAnson/markdownlint-cli2-action from 19 to 23 (#65)

Author: dependabot[bot]

.github/workflows/ci.yaml

@@ -12,7 +12,7 @@ jobs:
     steps:
       - uses: actions/checkout@v6
       - name: Lint Markdown
-        uses: DavidAnson/markdownlint-cli2-action@v19
+        uses: DavidAnson/markdownlint-cli2-action@v23
         with:
           globs: '**/*.md'
           config: 'config/custom.markdownlint.jsonc'

CLAUDE.md

@@ -60,6 +60,7 @@ Every `SKILL.md` follows this structure:
 - Provide complete, copy-pasteable snippets, not fragments
 - Reference packages by full name (e.g., `package:mocktail`)
 - Include anti-patterns alongside correct patterns when helpful
+- Align pipe characters vertically in all markdown tables (enforced by markdownlint MD060)
 
 ## Adding a New Skill
 

skills/bloc/SKILL.md

@@ -30,7 +30,7 @@ Apply these standards to ALL Bloc/Cubit work:
 
 | Aspect            | Cubit                          | Bloc                                    |
 | ----------------- | ------------------------------ | --------------------------------------- |
-| API               | Functions → `emit(state)`      | Events → `on<Event>` → `emit(state)`   |
+| API               | Functions → `emit(state)`      | Events → `on<Event>` → `emit(state)`    |
 | Complexity        | Low                            | Higher                                  |
 | Traceability      | Less (no event log)            | Full (events + transitions)             |
 | When to use       | Simple state, UI-driven logic  | Complex flows, event-driven, transforms |
@@ -76,13 +76,13 @@ class CounterBloc extends Bloc<CounterEvent, int> {
 
 **Pattern:** `BlocSubject` + `Noun` + `VerbPastTense`
 
-| Event class name              | Meaning                          |
-| ----------------------------- | -------------------------------- |
-| `TodoListSubscriptionRequested` | Subscribing to todo list stream |
-| `TodoListTodoDeleted`         | Deleting a specific todo         |
-| `TodoListUndoDeletionRequested` | Undoing the last deletion      |
-| `LoginFormSubmitted`          | Submitting the login form        |
-| `ProfilePageRefreshed`        | Refreshing the profile page      |
+| Event class name                | Meaning                          |
+| ------------------------------- | -------------------------------- |
+| `TodoListSubscriptionRequested` | Subscribing to todo list stream  |
+| `TodoListTodoDeleted`           | Deleting a specific todo         |
+| `TodoListUndoDeletionRequested` | Undoing the last deletion        |
+| `LoginFormSubmitted`            | Submitting the login form        |
+| `ProfilePageRefreshed`          | Refreshing the profile page      |
 
 ```dart
 sealed class TodoListEvent extends Equatable {

skills/bloc/references/patterns.md

@@ -60,7 +60,7 @@ class SearchBloc extends Bloc<SearchEvent, SearchState> {
 ```
 
 | Transformer       | Behavior                                              |
-| ------------------ | ---------------------------------------------------- |
+| ------------------| ----------------------------------------------------- |
 | `concurrent()`    | Process all events concurrently (default)             |
 | `sequential()`    | Process events one at a time in order                 |
 | `droppable()`     | Ignore new events while one is processing             |

skills/bloc/references/testing.md

@@ -4,14 +4,14 @@
 
 | Parameter  | Type                        | Purpose                                        |
 | ---------- | --------------------------- | ---------------------------------------------- |
-| `build`    | `() => Bloc/Cubit`         | Creates the Bloc/Cubit under test              |
-| `act`      | `(bloc) => void`           | Interacts with the Bloc/Cubit                  |
-| `expect`   | `() => List<State>`        | Expected states emitted (in order)             |
-| `seed`     | `() => State`              | Initial state before `act`                     |
-| `setUp`    | `() => void`               | Runs before `build`                            |
-| `verify`   | `(bloc) => void`           | Additional verifications after `expect`        |
-| `errors`   | `() => List<Matcher>`      | Expected errors thrown                         |
-| `wait`     | `Duration`                 | Wait duration before asserting (for debounce)  |
+| `build`    | `() => Bloc/Cubit`          | Creates the Bloc/Cubit under test              |
+| `act`      | `(bloc) => void`            | Interacts with the Bloc/Cubit                  |
+| `expect`   | `() => List<State>`         | Expected states emitted (in order)             |
+| `seed`     | `() => State`               | Initial state before `act`                     |
+| `setUp`    | `() => void`                | Runs before `build`                            |
+| `verify`   | `(bloc) => void`            | Additional verifications after `expect`        |
+| `errors`   | `() => List<Matcher>`       | Expected errors thrown                         |
+| `wait`     | `Duration`                  | Wait duration before asserting (for debounce)  |
 
 ## Cubit Test Example
 

skills/bloc/references/widgets.md

@@ -1,23 +1,23 @@
 # Flutter Widgets
 
-| Widget              | Purpose                                                    |
-| ------------------- | ---------------------------------------------------------- |
-| `BlocProvider`      | Creates and provides a Bloc/Cubit to the subtree           |
-| `BlocBuilder`       | Rebuilds widget when state changes                         |
+| Widget              | Purpose                                                      |
+| ------------------- | ------------------------------------------------------------ |
+| `BlocProvider`      | Creates and provides a Bloc/Cubit to the subtree             |
+| `BlocBuilder`       | Rebuilds widget when state changes                           |
 | `BlocListener`      | Executes side effects (navigation, snackbar) on state change |
-| `BlocConsumer`      | Combines `BlocBuilder` + `BlocListener`                    |
-| `BlocSelector`      | Rebuilds only when a selected property changes             |
-| `MultiBlocProvider` | Provides multiple Blocs/Cubits without nesting             |
-| `MultiBlocListener` | Registers multiple listeners without nesting               |
-| `RepositoryProvider`| Provides a repository to the subtree                       |
+| `BlocConsumer`      | Combines `BlocBuilder` + `BlocListener`                      |
+| `BlocSelector`      | Rebuilds only when a selected property changes               |
+| `MultiBlocProvider` | Provides multiple Blocs/Cubits without nesting               |
+| `MultiBlocListener` | Registers multiple listeners without nesting                 |
+| `RepositoryProvider`| Provides a repository to the subtree                         |
 
 ## Context Extensions
 
-| Extension          | Purpose                                          | Rebuilds? |
-| ------------------ | ------------------------------------------------ | --------- |
-| `context.read<T>()`   | Access Bloc/Cubit instance (one-time read)   | No        |
-| `context.watch<T>()`  | Access Bloc/Cubit and subscribe to changes   | Yes       |
-| `context.select<T, R>()` | Subscribe to a specific state property    | Yes (selective) |
+| Extension                | Purpose                                          | Rebuilds?       |
+| ------------------------ | ------------------------------------------------ | --------------- |
+| `context.read<T>()`      | Access Bloc/Cubit instance (one-time read)       | No              |
+| `context.watch<T>()`     | Access Bloc/Cubit and subscribe to changes       | Yes             |
+| `context.select<T, R>()` | Subscribe to a specific state property           | Yes (selective) |
 
 - Use `context.read` in callbacks (`onPressed`, `onTap`, `initState`)
 - Use `context.watch` or `BlocBuilder` in `build` methods

skills/internationalization/SKILL.md

@@ -23,10 +23,10 @@ Apply these standards to ALL internationalization work:
 
 ## Key Definitions
 
-| Term                          | Definition                                                                                  |
-| ----------------------------- | ------------------------------------------------------------------------------------------- |
-| **Locale**                    | Set of properties defining user region, language, and preferences (currency, time, numbers)  |
-| **Localization (l10n)**       | Process of adapting software for a specific language by translating text and adding regional layouts |
+| Term                            | Definition                                                                                            |
+| ------------------------------- | ----------------------------------------------------------------------------------------------------- |
+| **Locale**                      | Set of properties defining user region, language, and preferences (currency, time, numbers)           |
+| **Localization (l10n)**         | Process of adapting software for a specific language by translating text and adding regional layouts  |
 | **Internationalization (i18n)** | Process of designing software so it can be adapted to different languages without engineering changes |
 
 ## Setup Pipeline and ARB File Format
@@ -138,17 +138,17 @@ See [references/backend.md](references/backend.md) for multi-language content st
 
 ## Quick Reference
 
-| Package                    | Purpose                                |
-| -------------------------- | -------------------------------------- |
-| `flutter_localizations`    | Flutter's built-in localization support |
-| `intl`                     | Internationalization utilities          |
+| Package                 | Purpose                                 |
+| ----------------------- | --------------------------------------- |
+| `flutter_localizations` | Flutter's built-in localization support |
+| `intl`                  | Internationalization utilities          |
 
-| Command              | Purpose                                     |
-| -------------------- | ------------------------------------------- |
-| `flutter gen-l10n`   | Generate localization classes from ARB files |
+| Command            | Purpose                                      |
+| ------------------ | -------------------------------------------- |
+| `flutter gen-l10n` | Generate localization classes from ARB files |
 
-| File               | Purpose                                          |
-| ------------------ | ------------------------------------------------ |
+| File               | Purpose                                            |
+| ------------------ | -------------------------------------------------- |
 | `l10n.yaml`        | Localization configuration (ARB dir, output, etc.) |
-| `app_en.arb`       | Template ARB file (source of truth)              |
-| `app_<locale>.arb` | Translated ARB file for each locale              |
+| `app_en.arb`       | Template ARB file (source of truth)                |
+| `app_<locale>.arb` | Translated ARB file for each locale                |

skills/internationalization/references/directionality.md

@@ -19,10 +19,10 @@ Retrieve the current direction: `Directionality.of(context)`
 
 ## Visual vs Directional Widgets
 
-| Widget Type     | Direction Terms       | Use Case                                    |
-| --------------- | --------------------- | ------------------------------------------- |
-| **Visual**      | top, left, right, bottom | Absolute positioning that never changes    |
-| **Directional** | top, start, end, bottom  | Relative to text direction (respects RTL)  |
+| Widget Type     | Direction Terms            | Use Case                                  |
+| --------------- | -------------------------- | ----------------------------------------- |
+| **Visual**      | top, left, right, bottom   | Absolute positioning that never changes   |
+| **Directional** | top, start, end, bottom    | Relative to text direction (respects RTL) |
 
 ## `EdgeInsetsDirectional` vs `EdgeInsets`
 

skills/material-theming/SKILL.md

@@ -162,12 +162,12 @@ See [references/spacing.md](references/spacing.md) for the full `AppSpacing` cla
 | `cardTheme`               | Card default styling                         |
 | `dialogTheme`             | Dialog default styling                       |
 
-| Material 3 Color Role | Typical Use                          |
-| ---------------------- | ------------------------------------ |
-| `primary`              | Key UI elements, FAB, active states  |
-| `onPrimary`            | Text/icons on primary color          |
-| `secondary`            | Less prominent UI elements           |
-| `surface`              | Card, sheet, dialog backgrounds      |
-| `onSurface`            | Text/icons on surface color          |
-| `error`                | Error indicators, destructive actions |
-| `outline`              | Borders, dividers                    |
+| Material 3 Color Role | Typical Use                           |
+| --------------------- | ------------------------------------- |
+| `primary`             | Key UI elements, FAB, active states   |
+| `onPrimary`           | Text/icons on primary color           |
+| `secondary`           | Less prominent UI elements            |
+| `surface`             | Card, sheet, dialog backgrounds       |
+| `onSurface`           | Text/icons on surface color           |
+| `error`               | Error indicators, destructive actions |
+| `outline`             | Borders, dividers                     |

skills/navigation/SKILL.md

@@ -116,9 +116,9 @@ class FlutterPageRoute extends GoRouteData {
 
 ### `go()` vs `push()`
 
-| Method   | URL Updates | Back Button | Use Case                                    |
-| -------- | ----------- | ----------- | ------------------------------------------- |
-| `go()`   | Yes         | App bar     | Standard navigation between screens         |
+| Method   | URL Updates | Back Button | Use Case                                     |
+| -------- | ----------- | ----------- | -------------------------------------------- |
+| `go()`   | Yes         | App bar     | Standard navigation between screens          |
 | `push()` | No          | System      | When expecting return data from popped route |
 
 ### Using `go()` (Default)
@@ -210,12 +210,12 @@ class AppShellRoute extends ShellRouteData {
 
 ## Quick Reference
 
-| Package              | Purpose                                      |
-| -------------------- | -------------------------------------------- |
-| `go_router`          | Declarative routing built on Navigator 2.0   |
-| `go_router_builder`  | Code generation for type-safe route classes   |
+| Package             | Purpose                                     |
+| ------------------- | ------------------------------------------- |
+| `go_router`         | Declarative routing built on Navigator 2.0  |
+| `go_router_builder` | Code generation for type-safe route classes |
 
 | Command                                                    | Purpose                          |
 | ---------------------------------------------------------- | -------------------------------- |
-| `dart run build_runner build --delete-conflicting-outputs`  | Generate type-safe route helpers |
-| `dart run build_runner watch --delete-conflicting-outputs`  | Watch and regenerate on changes  |
+| `dart run build_runner build --delete-conflicting-outputs` | Generate type-safe route helpers |
+| `dart run build_runner watch --delete-conflicting-outputs` | Watch and regenerate on changes  |