skilldrop-cli 0.16.29 → 0.16.30

package/packs/dev-team/skills/codebase-tour/evals/files/1/openapi.yaml

@@ -0,0 +1,21 @@
+openapi: 3.0.3
+info:
+  title: shipwright-api
+  version: 2.4.0
+paths:
+  /shipments:
+    get:
+      summary: List shipments
+      parameters:
+        - name: status
+          in: query
+          schema:
+            type: string
+            enum: [pending, in_transit, delivered]
+        - name: limit
+          in: query
+          schema:
+            type: integer
+      responses:
+        "200":
+          description: A page of shipments

package/packs/dev-team/skills/codebase-tour/evals/files/1/package-lock.json

@@ -0,0 +1,7 @@
+{
+  "name": "shipwright-api",
+  "version": "2.4.0",
+  "lockfileVersion": 3,
+  "requires": true,
+  "packages": {}
+}

package/packs/dev-team/skills/codebase-tour/evals/files/1/package.json

@@ -0,0 +1,29 @@
+{
+  "name": "shipwright-api",
+  "version": "2.4.0",
+  "private": true,
+  "type": "module",
+  "main": "dist/server.js",
+  "scripts": {
+    "build": "tsc -p tsconfig.json",
+    "start": "node dist/server.js",
+    "start:watch": "tsx watch src/server.ts",
+    "lint": "eslint src tests",
+    "typecheck": "tsc --noEmit",
+    "test": "vitest run",
+    "gen:types": "openapi-typescript openapi.yaml -o src/generated/api-types.ts"
+  },
+  "dependencies": {
+    "express": "^4.19.2",
+    "pg": "^8.11.3"
+  },
+  "devDependencies": {
+    "@types/express": "^4.17.21",
+    "eslint": "^8.57.0",
+    "openapi-typescript": "^6.7.4",
+    "supertest": "^6.3.4",
+    "tsx": "^4.7.1",
+    "typescript": "^5.4.2",
+    "vitest": "^1.4.0"
+  }
+}

package/packs/dev-team/skills/codebase-tour/evals/files/1/src/config.ts

@@ -0,0 +1,20 @@
+export interface Config {
+  port: number;
+  databaseUrl: string;
+  maxPageSize: number;
+  legacySort: boolean;
+}
+
+export function loadConfig(): Config {
+  const databaseUrl = process.env.DATABASE_URL;
+  if (!databaseUrl) {
+    throw new Error("DATABASE_URL is required");
+  }
+  return {
+    port: Number(process.env.PORT ?? 3000),
+    databaseUrl,
+    maxPageSize: Number(process.env.MAX_PAGE_SIZE ?? 100),
+    // Old consoles sort by created_at ascending; remove once v1 console is retired.
+    legacySort: process.env.SHIPMENTS_LEGACY_SORT === "1",
+  };
+}

package/packs/dev-team/skills/codebase-tour/evals/files/1/src/domain/shipments/filter.ts

@@ -0,0 +1,24 @@
+import type { Config } from "../../config.js";
+import type { ListShipmentsQuery } from "../../generated/api-types.js";
+
+export interface ShipmentFilter {
+  where: string[];
+  params: unknown[];
+  orderBy: string;
+  limit: number;
+}
+
+export function buildShipmentFilter(query: ListShipmentsQuery, config: Config): ShipmentFilter {
+  const where: string[] = [];
+  const params: unknown[] = [];
+  if (query.status) {
+    params.push(query.status);
+    where.push(`status = $${params.length}`);
+  }
+  return {
+    where,
+    params,
+    orderBy: config.legacySort ? "created_at ASC" : "created_at DESC",
+    limit: query.limit ?? 50,
+  };
+}

package/packs/dev-team/skills/codebase-tour/evals/files/1/src/domain/shipments/repository.ts

@@ -0,0 +1,15 @@
+import pg from "pg";
+import type { Config } from "../../config.js";
+import type { ShipmentFilter } from "./filter.js";
+
+export async function queryShipments(config: Config, filter: ShipmentFilter) {
+  const client = new pg.Client({ connectionString: config.databaseUrl });
+  await client.connect();
+  try {
+    const where = filter.where.length ? "WHERE " + filter.where.join(" AND ") : "";
+    const sql = `SELECT id, status, carrier, created_at FROM shipments ${where} ORDER BY ${filter.orderBy} LIMIT ${filter.limit}`;
+    return (await client.query(sql, filter.params)).rows;
+  } finally {
+    await client.end();
+  }
+}

package/packs/dev-team/skills/codebase-tour/evals/files/1/src/errors.ts

@@ -0,0 +1,9 @@
+export class HttpError extends Error {
+  constructor(public status: number, public code: string, message: string) {
+    super(message);
+  }
+}
+
+export function badRequest(code: string, message: string): HttpError {
+  return new HttpError(400, code, message);
+}

package/packs/dev-team/skills/codebase-tour/evals/files/1/src/generated/api-types.ts

@@ -0,0 +1,9 @@
+/**
+ * This file was auto-generated by openapi-typescript.
+ * Do not make direct changes to the file.
+ */
+
+export interface ListShipmentsQuery {
+  status?: "pending" | "in_transit" | "delivered";
+  limit?: number;
+}

package/packs/dev-team/skills/codebase-tour/evals/files/1/src/http/handlers/shipments.ts

@@ -0,0 +1,38 @@
+import type { Request, Response, NextFunction } from "express";
+import type { Config } from "../../config.js";
+import type { ListShipmentsQuery } from "../../generated/api-types.js";
+import { badRequest } from "../../errors.js";
+import { buildShipmentFilter } from "../../domain/shipments/filter.js";
+import { queryShipments } from "../../domain/shipments/repository.js";
+
+const STATUSES = ["pending", "in_transit", "delivered"];
+
+export function parseListQuery(raw: Record<string, unknown>, config: Config): ListShipmentsQuery {
+  const query: ListShipmentsQuery = {};
+  if (raw.status !== undefined) {
+    if (typeof raw.status !== "string" || !STATUSES.includes(raw.status)) {
+      throw badRequest("invalid_status", "status must be one of " + STATUSES.join(", "));
+    }
+    query.status = raw.status as ListShipmentsQuery["status"];
+  }
+  if (raw.limit !== undefined) {
+    const limit = Number(raw.limit);
+    if (!Number.isInteger(limit) || limit < 1 || limit > config.maxPageSize) {
+      throw badRequest("invalid_limit", "limit must be 1.." + config.maxPageSize);
+    }
+    query.limit = limit;
+  }
+  return query;
+}
+
+export function listShipments(config: Config) {
+  return async (req: Request, res: Response, next: NextFunction) => {
+    try {
+      const query = parseListQuery(req.query, config);
+      const filter = buildShipmentFilter(query, config);
+      res.json({ items: await queryShipments(config, filter) });
+    } catch (err) {
+      next(err);
+    }
+  };
+}

package/packs/dev-team/skills/codebase-tour/evals/files/1/src/http/routes.ts

@@ -0,0 +1,7 @@
+import type { Express } from "express";
+import type { Config } from "../config.js";
+import { listShipments } from "./handlers/shipments.js";
+
+export function registerRoutes(app: Express, config: Config): void {
+  app.get("/shipments", listShipments(config));
+}

package/packs/dev-team/skills/codebase-tour/evals/files/1/src/jobs/export-shipments.ts

@@ -0,0 +1,22 @@
+// Nightly CSV export for the finance team. Runs from the scheduler, not the API.
+import pg from "pg";
+import { loadConfig } from "../config.js";
+
+export async function exportShipments(status?: string): Promise<string> {
+  const config = loadConfig();
+  const client = new pg.Client({ connectionString: config.databaseUrl });
+  await client.connect();
+  try {
+    // Same filtering as the list endpoint, kept here so the job has no HTTP dependency.
+    const params: unknown[] = [];
+    let where = "";
+    if (status) {
+      params.push(status);
+      where = "WHERE status = $1";
+    }
+    const rows = (await client.query(`SELECT id, status, carrier FROM shipments ${where} ORDER BY created_at DESC`, params)).rows;
+    return rows.map((r) => [r.id, r.status, r.carrier].join(",")).join("\n");
+  } finally {
+    await client.end();
+  }
+}

package/packs/dev-team/skills/codebase-tour/evals/files/1/src/server.ts

@@ -0,0 +1,18 @@
+import express from "express";
+import { loadConfig } from "./config.js";
+import { registerRoutes } from "./http/routes.js";
+import { HttpError } from "./errors.js";
+
+const config = loadConfig();
+const app = express();
+registerRoutes(app, config);
+
+app.use((err: unknown, _req: express.Request, res: express.Response, _next: express.NextFunction) => {
+  if (err instanceof HttpError) {
+    res.status(err.status).json({ error: err.code, message: err.message });
+    return;
+  }
+  res.status(500).json({ error: "internal" });
+});
+
+app.listen(config.port);

package/packs/dev-team/skills/codebase-tour/evals/files/1/tests/domain/filter.test.ts

@@ -0,0 +1,16 @@
+import { describe, it, expect } from "vitest";
+import { buildShipmentFilter } from "../../src/domain/shipments/filter.js";
+
+const config = { port: 0, databaseUrl: "postgres://unused", maxPageSize: 100, legacySort: false };
+
+describe("buildShipmentFilter", () => {
+  it("adds a status clause with a positional parameter", () => {
+    const f = buildShipmentFilter({ status: "pending" }, config);
+    expect(f.where).toEqual(["status = $1"]);
+    expect(f.params).toEqual(["pending"]);
+  });
+
+  it("sorts ascending when legacySort is on", () => {
+    expect(buildShipmentFilter({}, { ...config, legacySort: true }).orderBy).toBe("created_at ASC");
+  });
+});

package/packs/dev-team/skills/codebase-tour/evals/files/1/tests/http/shipments.test.ts

@@ -0,0 +1,18 @@
+import { describe, it, expect } from "vitest";
+import { parseListQuery } from "../../src/http/handlers/shipments.js";
+
+const config = { port: 0, databaseUrl: "postgres://unused", maxPageSize: 100, legacySort: false };
+
+describe("parseListQuery", () => {
+  it("accepts a known status", () => {
+    expect(parseListQuery({ status: "delivered" }, config)).toEqual({ status: "delivered" });
+  });
+
+  it("rejects an unknown status with invalid_status", () => {
+    expect(() => parseListQuery({ status: "lost" }, config)).toThrowError(/status must be one of/);
+  });
+
+  it("rejects a limit above maxPageSize", () => {
+    expect(() => parseListQuery({ limit: "500" }, config)).toThrowError(/limit must be/);
+  });
+});

package/packs/dev-team/skills/codebase-tour/evals/files/1/tsconfig.json

@@ -0,0 +1,10 @@
+{
+  "compilerOptions": {
+    "target": "ES2022",
+    "module": "NodeNext",
+    "moduleResolution": "NodeNext",
+    "outDir": "dist",
+    "strict": true
+  },
+  "include": ["src"]
+}

package/packs/dev-team/skills/codebase-tour/evals/files/1/vitest.config.ts

@@ -0,0 +1,5 @@
+import { defineConfig } from "vitest/config";
+
+export default defineConfig({
+  test: { include: ["tests/**/*.test.ts"] },
+});

package/packs/dev-team/skills/codebase-tour/evals/files/2/.github/workflows/ci.yml

@@ -0,0 +1,16 @@
+name: ci
+on: [push, pull_request]
+jobs:
+  check:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+        with:
+          fetch-depth: 0
+      - uses: actions/setup-python@v5
+        with:
+          python-version: |
+            3.10
+            3.12
+      - run: pip install nox
+      - run: nox -s lint tests

package/packs/dev-team/skills/codebase-tour/evals/files/2/README.md

@@ -0,0 +1,16 @@
+# ledgerline
+
+Turns bank-statement CSV exports into one ledger format.
+
+## Install
+
+    pip install -e .
+
+## Running the tests
+
+    python setup.py test
+
+## Adding a bank
+
+Drop a new module in `src/ledgerline/parsers/` and it is picked up automatically.
+Each parser subclasses `BaseParser` and implements `parse_row`.

package/packs/dev-team/skills/codebase-tour/evals/files/2/noxfile.py

@@ -0,0 +1,14 @@
+import nox
+
+
+@nox.session(python=["3.10", "3.12"])
+def tests(session):
+    session.install("-e", ".[dev]")
+    session.run("pytest", "-m", "not slow", *session.posargs)
+
+
+@nox.session
+def lint(session):
+    session.install("ruff")
+    session.run("ruff", "check", "src", "tests")
+    session.run("ruff", "format", "--check", "src", "tests")

package/packs/dev-team/skills/codebase-tour/evals/files/2/pyproject.toml

@@ -0,0 +1,22 @@
+[build-system]
+requires = ["setuptools>=68", "setuptools-scm>=8"]
+build-backend = "setuptools.build_meta"
+
+[project]
+name = "ledgerline"
+dynamic = ["version"]
+requires-python = ">=3.10"
+dependencies = []
+
+[project.optional-dependencies]
+dev = ["pytest>=8", "ruff>=0.4", "nox>=2024.3"]
+
+[project.scripts]
+ledgerline = "ledgerline.cli:main"
+
+[tool.setuptools_scm]
+version_file = "src/ledgerline/_version.py"
+
+[tool.pytest.ini_options]
+testpaths = ["tests"]
+markers = ["slow: reads the large sample statements"]

package/packs/dev-team/skills/codebase-tour/evals/files/2/src/ledgerline/__init__.py

@@ -0,0 +1,3 @@
+from ledgerline._version import __version__
+
+__all__ = ["__version__"]

package/packs/dev-team/skills/codebase-tour/evals/files/2/src/ledgerline/_version.py

@@ -0,0 +1,4 @@
+# file generated by setuptools-scm
+# don't change, don't track in version control
+__version__ = version = "1.6.2.dev4"
+__version_tuple__ = version_tuple = (1, 6, 2, "dev4")

package/packs/dev-team/skills/codebase-tour/evals/files/2/src/ledgerline/cli.py

@@ -0,0 +1,32 @@
+import argparse
+import csv
+import os
+import sys
+
+from ledgerline.detect import detect_bank
+from ledgerline.errors import ParseError
+from ledgerline.parsers import PARSERS
+
+
+def main(argv=None):
+    ap = argparse.ArgumentParser(prog="ledgerline")
+    ap.add_argument("statement")
+    ap.add_argument("--bank", help="skip detection and use this parser")
+    args = ap.parse_args(argv)
+    currency = os.environ.get("LEDGERLINE_CURRENCY", "GBP")
+    with open(args.statement, newline="", encoding="utf-8-sig") as fh:
+        rows = list(csv.reader(fh))
+    bank = args.bank or detect_bank(rows[0])
+    parser = PARSERS[bank](currency=currency)
+    writer = csv.writer(sys.stdout)
+    for n, row in enumerate(rows[1:], start=2):
+        try:
+            writer.writerow(parser.parse_row(row).as_csv())
+        except ParseError as e:
+            print(f"line {n}: {e}", file=sys.stderr)
+            return 1
+    return 0
+
+
+if __name__ == "__main__":
+    sys.exit(main())

package/packs/dev-team/skills/codebase-tour/evals/files/2/src/ledgerline/detect.py

@@ -0,0 +1,15 @@
+from ledgerline.errors import ParseError
+
+# Header row of each bank's export -> parser name in PARSERS.
+HEADER_SIGNATURES = {
+    ("Date", "Description", "Amount", "Balance"): "acme",
+    ("Posted", "Payee", "Debit", "Credit"): "harbor",
+}
+
+
+def detect_bank(header):
+    key = tuple(cell.strip() for cell in header)
+    try:
+        return HEADER_SIGNATURES[key]
+    except KeyError:
+        raise ParseError(f"unrecognised header {list(key)}; pass --bank")

package/packs/dev-team/skills/codebase-tour/evals/files/2/src/ledgerline/errors.py

@@ -0,0 +1,2 @@
+class ParseError(ValueError):
+    """A statement row that cannot be turned into an Entry. The message names the column."""

package/packs/dev-team/skills/codebase-tour/evals/files/2/src/ledgerline/money.py

@@ -0,0 +1,14 @@
+from decimal import Decimal, InvalidOperation
+
+from ledgerline.errors import ParseError
+
+
+def parse_amount(text: str) -> Decimal:
+    """Parse '1,234.50' or '(12.00)' into a Decimal. Parentheses mean a debit."""
+    cleaned = text.strip().replace(",", "")
+    negative = cleaned.startswith("(") and cleaned.endswith(")")
+    try:
+        value = Decimal(cleaned.strip("()"))
+    except InvalidOperation:
+        raise ParseError(f"amount {text!r} is not a number")
+    return -value if negative else value

package/packs/dev-team/skills/codebase-tour/evals/files/2/src/ledgerline/parsers/__init__.py

@@ -0,0 +1,8 @@
+from ledgerline.parsers.acme import AcmeParser
+from ledgerline.parsers.harbor import HarborParser
+
+# Explicit registry: a parser module is not used until it is listed here.
+PARSERS = {
+    "acme": AcmeParser,
+    "harbor": HarborParser,
+}

package/packs/dev-team/skills/codebase-tour/evals/files/2/src/ledgerline/parsers/acme.py

@@ -0,0 +1,18 @@
+from datetime import datetime
+
+from ledgerline.errors import ParseError
+from ledgerline.money import parse_amount
+from ledgerline.parsers.base import BaseParser, Entry
+
+
+class AcmeParser(BaseParser):
+    date_format = "%d/%m/%Y"
+
+    def parse_row(self, row):
+        if len(row) != 4:
+            raise ParseError(f"expected 4 columns, got {len(row)}")
+        try:
+            on = datetime.strptime(row[0], self.date_format).date()
+        except ValueError:
+            raise ParseError(f"Date {row[0]!r} is not DD/MM/YYYY")
+        return Entry(on=on, payee=row[1].strip(), amount=parse_amount(row[2]), currency=self.currency)

package/packs/dev-team/skills/codebase-tour/evals/files/2/src/ledgerline/parsers/base.py

@@ -0,0 +1,24 @@
+from dataclasses import dataclass
+from datetime import date
+from decimal import Decimal
+
+
+@dataclass(frozen=True)
+class Entry:
+    on: date
+    payee: str
+    amount: Decimal
+    currency: str
+
+    def as_csv(self):
+        return [self.on.isoformat(), self.payee, f"{self.amount:.2f}", self.currency]
+
+
+class BaseParser:
+    date_format = "%Y-%m-%d"
+
+    def __init__(self, currency: str):
+        self.currency = currency
+
+    def parse_row(self, row) -> Entry:
+        raise NotImplementedError

package/packs/dev-team/skills/codebase-tour/evals/files/2/src/ledgerline/parsers/harbor.py

@@ -0,0 +1,13 @@
+from datetime import datetime
+from decimal import Decimal
+
+from ledgerline.parsers.base import BaseParser, Entry
+
+
+class HarborParser(BaseParser):
+    def parse_row(self, row):
+        on = datetime.strptime(row[0], self.date_format).date()
+        # Harbor splits debit and credit into two columns.
+        debit = Decimal(row[2].replace(",", "") or "0")
+        credit = Decimal(row[3].replace(",", "") or "0")
+        return Entry(on=on, payee=row[1].strip(), amount=credit - debit, currency=self.currency)

package/packs/dev-team/skills/codebase-tour/evals/files/2/tests/fixtures/acme_sample.csv

@@ -0,0 +1,3 @@
+Date,Description,Amount,Balance
+02/03/2026,Corner Bakery,4.20,995.80
+03/03/2026,Refund Northwind Books,(12.00),983.80

package/packs/dev-team/skills/codebase-tour/evals/files/2/tests/test_acme.py

@@ -0,0 +1,23 @@
+from datetime import date
+from decimal import Decimal
+from pathlib import Path
+import csv
+
+import pytest
+
+from ledgerline.errors import ParseError
+from ledgerline.parsers.acme import AcmeParser
+
+FIXTURES = Path(__file__).parent / "fixtures"
+
+
+def test_parses_sample_rows():
+    rows = list(csv.reader(open(FIXTURES / "acme_sample.csv", encoding="utf-8")))
+    entries = [AcmeParser(currency="GBP").parse_row(r) for r in rows[1:]]
+    assert entries[0].on == date(2026, 3, 2)
+    assert entries[1].amount == Decimal("-12.00")
+
+
+def test_bad_date_names_the_column():
+    with pytest.raises(ParseError, match="Date"):
+        AcmeParser(currency="GBP").parse_row(["2026-03-02", "Cafe", "3.10", "0"])

package/packs/dev-team/skills/codebase-tour/evals/files/2/tests/test_detect.py

@@ -0,0 +1,13 @@
+import pytest
+
+from ledgerline.detect import detect_bank
+from ledgerline.errors import ParseError
+
+
+def test_detects_acme_header():
+    assert detect_bank(["Date", "Description", "Amount", "Balance"]) == "acme"
+
+
+def test_unknown_header_tells_the_user_to_pass_bank():
+    with pytest.raises(ParseError, match="--bank"):
+        detect_bank(["When", "What", "How much"])

package/packs/dev-team/skills/codebase-tour/evals/files/3/.github/workflows/ci.yml

@@ -0,0 +1,28 @@
+name: ci
+on: [push, pull_request]
+jobs:
+  unit:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-go@v5
+        with:
+          go-version: "1.22"
+      - run: make lint
+      - run: make test
+  integration:
+    runs-on: ubuntu-latest
+    services:
+      postgres:
+        image: postgres:16
+        env:
+          POSTGRES_PASSWORD: example-only
+        ports: ["5432:5432"]
+    env:
+      TIDEPOOL_DATABASE_URL: postgres://postgres:example-only@localhost:5432/postgres?sslmode=disable
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-go@v5
+        with:
+          go-version: "1.22"
+      - run: make test-integration

package/packs/dev-team/skills/codebase-tour/evals/files/3/Makefile

@@ -0,0 +1,16 @@
+.PHONY: build test test-integration generate lint
+
+build:
+	go build -o bin/tidepool ./cmd/tidepool
+
+test:
+	go test ./...
+
+test-integration:
+	go test -tags=integration ./internal/store/...
+
+generate:
+	sqlc generate
+
+lint:
+	golangci-lint run

package/packs/dev-team/skills/codebase-tour/evals/files/3/README.md

@@ -0,0 +1,16 @@
+# tidepool
+
+Delivers webhook events to tenant endpoints, with retries.
+
+## Configuration
+
+Retries are set in `config.yaml`:
+
+```yaml
+max_attempts: 5
+retry_delay: 2s
+```
+
+## Development
+
+`make test` runs the full test suite, including the database tests.

package/packs/dev-team/skills/codebase-tour/evals/files/3/cmd/tidepool/main.go

@@ -0,0 +1,24 @@
+package main
+
+import (
+	"context"
+	"log"
+	"net/http"
+
+	"example.com/tidepool/internal/config"
+	"example.com/tidepool/internal/dispatch"
+	"example.com/tidepool/internal/store"
+)
+
+func main() {
+	cfg, err := config.FromEnv()
+	if err != nil {
+		log.Fatalf("config: %v", err)
+	}
+	db, err := store.Open(context.Background(), cfg.DatabaseURL)
+	if err != nil {
+		log.Fatalf("store: %v", err)
+	}
+	d := dispatch.New(cfg, store.New(db), dispatch.NewHTTPSender(http.DefaultClient))
+	log.Fatal(d.Run(context.Background()))
+}

package/packs/dev-team/skills/codebase-tour/evals/files/3/go.mod

@@ -0,0 +1,5 @@
+module example.com/tidepool
+
+go 1.22
+
+require github.com/jackc/pgx/v5 v5.5.5