skilldrop-cli 0.16.29 → 0.16.31

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

@@ -0,0 +1,77 @@
+{
+  "skill_name": "codebase-tour",
+  "evals": [
+    {
+      "id": 1,
+      "prompt": "I'm new to this repo (it's the current directory) and my first ticket is to add a `carrier` filter to GET /shipments, so dispatchers can see one carrier's shipments at a time. Get me oriented before I start: how to run it, where the change goes, and anything that will bite me.",
+      "assertions": [
+        "Opens with a short list of the few things that matter most for this change, before any section, and does not open with a directory listing",
+        "Every command given is cited to package.json or .github/workflows/ci.yml and marked as either run or only read; `npm run dev` is not given as a way to run the server",
+        "Says where the README and the code disagree: there is no `dev` script (package.json has `start:watch`), there is no `src/routes/` (routes are registered in src/http/routes.ts), and config comes from env vars in src/config.ts, not config/default.json",
+        "Says the change goes in `parseListQuery` (src/http/handlers/shipments.ts) and `buildShipmentFilter` (src/domain/shipments/filter.ts), and that openapi.yaml needs the new query parameter",
+        "Names src/generated/api-types.ts as generated from openapi.yaml by `npm run gen:types`, says not to edit it by hand, and notes that CI fails if it drifts (.github/workflows/ci.yml)",
+        "Names src/jobs/export-shipments.ts as a second copy of the shipment filtering that the change does not reach on its own, notes it has no tests, and raises whether it needs the carrier filter too",
+        "Names tests/http/shipments.test.ts and tests/domain/filter.test.ts as the tests for the area, with a way to run just those through the repo's vitest script, and notes DATABASE_URL is required to start the server (src/config.ts)",
+        "Does not invent commands, files, scripts or tools that are not in the repo"
+      ]
+    },
+    {
+      "id": 2,
+      "prompt": "Never touched this codebase before. I need to add support for Northwind Credit Union's CSV export (header: Transaction Date, Merchant, Amount; dates like 2026-03-02). Where do I start, and how do I run the tests? The repo is the folder you're in.",
+      "assertions": [
+        "Says the README is wrong that a new parser module is picked up automatically: the PARSERS registry in src/ledgerline/parsers/__init__.py has to list it",
+        "Names HEADER_SIGNATURES in src/ledgerline/detect.py as a second place the new bank must be added, or detection fails for the new header",
+        "Gives the test command from noxfile.py or CI (`nox -s tests`, `nox -s lint tests`), says `python setup.py test` from the README cannot work because there is no setup.py, and notes the tests session skips tests marked `slow`",
+        "Gives a way to run a single test file through the runner the repo already uses (nox posargs or pytest), labelled as derived from the repo's config rather than presented as a defined command",
+        "Points to AcmeParser as the parser to copy and flags that HarborParser has no tests and does not use parse_amount or raise ParseError like the others",
+        "Identifies src/ledgerline/_version.py as generated by setuptools-scm and not to be edited",
+        "Each convention it states (ParseError naming the column, money.parse_amount for amounts, CSV samples in tests/fixtures/) is cited to a file in the repo",
+        "Does not invent commands, files, scripts or tools that are not in the repo"
+      ]
+    },
+    {
+      "id": 3,
+      "prompt": "I've got a ticket to switch webhook delivery retries in this repo to exponential backoff. I don't really know Go and I've never seen this code. Write me an orientation to docs/backoff-tour.md and don't change anything else.",
+      "assertions": [
+        "Writes docs/backoff-tour.md and modifies no other file in the repo",
+        "Says the README is wrong about config.yaml: there is no such file, and retry settings are the env vars TIDEPOOL_MAX_ATTEMPTS and TIDEPOOL_RETRY_DELAY read in internal/config/config.go",
+        "Says the README is wrong that `make test` includes the database tests: they sit behind the `integration` build tag and run with `make test-integration`, which needs Postgres and TIDEPOOL_DATABASE_URL",
+        "Names `Deliver` in internal/dispatch/dispatcher.go as where the change goes, and the injectable `sleep` field used by internal/dispatch/dispatcher_test.go as the way to test the delays",
+        "Names internal/replay/replay.go as a second retry loop with its own fixed delay that the change must also update or deliberately leave, and notes it has no tests",
+        "Names internal/store/query.sql.go as generated by sqlc from query.sql, regenerated with `make generate`, and not to be edited by hand",
+        "Every command is cited to the Makefile or .github/workflows/ci.yml and marked as run or only read; it does not claim to have run the integration tests",
+        "Does not invent commands, files, scripts or tools that are not in the repo"
+      ]
+    },
+    {
+      "id": 4,
+      "held_out": "2026-10-20",
+      "prompt": "Hi! I joined the checkout team on Monday and I've been handed ticket SHIP-214: \"Raise the EU free-shipping threshold from EUR 50 to EUR 60. UK and US stay as they are.\" I'm new to this repo and haven't used FastAPI much. Before I start, can you get me oriented: how do I install, run and test this thing, and where exactly would I make the change?",
+      "assertions": [
+        "Gives install/run/test commands taken from the Makefile and .github/workflows/ci.yml (`make install` / `pip install -e \".[dev]\"`, `make run` / `uvicorn shipcalc.main:app`, `make test` / pytest) and explicitly says the README's `pip install -r requirements.txt`, `flask run` and `python -m unittest discover` are stale (there is no requirements.txt and the app is FastAPI, not Flask).",
+        "Says the EU threshold's source of truth is data/rates.csv (line 2, `free_threshold` column), that src/shipcalc/generated/rates_table.py is generated (DO NOT EDIT header on line 1) and must not be hand-edited, and that the change is: edit the CSV, then run `make gen`; it notes that CI's `make check-generated` step fails if the generated file is out of sync with the CSV.",
+        "Finds the second, hard-coded copy of the threshold in src/shipcalc/api/preview.py line 5 (`FREE_SHIPPING_MIN`, used by the /preview/banner route) and says it must also change to 60, or the cart banner and the actual quote will disagree.",
+        "Traces the quote path with file:line citations: the /quote handler in src/shipcalc/api/routes.py (shipping_cost call at line 24) -> shipping_cost in src/shipcalc/shipping.py (line 13) -> the threshold comparison against RATES[region][\"free_threshold\"] at shipping.py line 15.",
+        "Flags the feature flag SHIPCALC_FF_FREE_SHIPPING read in src/shipcalc/shipping.py line 10 (free shipping applies only when it is exactly \"on\", and it is not in .env.example or the Makefile run target), so a local manual check of the new threshold will show no change unless the flag is set; also notes SHIPCALC_API_KEY is required at startup (src/shipcalc/config.py line 12).",
+        "Points out that shipping.py and preview.py have no tests while discounts and tax do (tests/unit/test_discounts.py, tests/unit/test_tax.py), and recommends adding boundary tests (e.g. 59.99 vs 60.00 for EU, plus UK/US unchanged) that follow the existing style: pytest functions, Decimal money values, and the flag set via monkeypatch or the environment.",
+        "Ends with one concrete first step whose result can be checked (e.g. run `make test` and see it pass before changing anything, or edit the CSV, run `make gen` and confirm `git diff` touches only the EU line of rates_table.py), and keeps the tour focused on this ticket instead of walking through tax and discounts in detail.",
+        "Invents no commands, files, functions or settings that are not in the repo (e.g. no requirements.txt, no `python -m shipcalc` or `flask run` as a working command, no existing test_shipping.py, no config value for the threshold outside rates.csv/preview.py)."
+      ]
+    },
+    {
+      "id": 5,
+      "held_out": "2026-10-20",
+      "prompt": "First week here, and my ticket is CAB-112: \"Let guests book stays of up to 21 nights (currently capped at 14).\" I'm new to this codebase. Can you walk me through the code for the booking flow, tell me where the change goes, and how I run and test it? I really don't want to break anything on my first PR.",
+      "assertions": [
+        "Gives commands from the root package.json and .github/workflows/ci.yml: `npm install`, `npm test` (which runs `node --test` in each workspace), `npm run gen` / `npm run gen:check`, and `npm run dev:api` (which reads ../../.env, so .env must be copied from .env.example first). It says the README's `yarn install`, `cd api && yarn start`, `yarn test` and the `api/config.js` reference are stale: the repo uses npm workspaces under packages/ and there is no api/ directory or config.js.",
+        "Identifies packages/api/src/generated/schema.js as generated (DO NOT EDIT on line 1, written by scripts/gen-schema.mjs) and says to change the nights `maximum` in packages/api/openapi.json (line 24) and run `npm run gen`, noting that CI's `npm run gen:check` fails if schema.js is hand-edited or stale.",
+        "Lists every copy of the 14-night limit that must move to 21: packages/api/openapi.json line 24 (via the regenerated schema.js), packages/api/src/services/bookings.js line 6 (`MAX_NIGHTS`), and packages/web/src/bookingForm.js line 2 (`MAX_NIGHTS`, the client-side check). It warns that changing only one of them gives a confusing 400 or 422, or a form that blocks valid stays. The README line also needs updating.",
+        "Traces the POST /bookings request path with file:line citations: packages/api/src/server.js mounts the router -> packages/api/src/routes/bookings.js line 7 runs `validate('BookingRequest')` -> packages/api/src/middleware/validate.js line 15 enforces `maximum` from the generated schema (400) -> createBooking / maxNightsFor in packages/api/src/services/bookings.js enforces MAX_NIGHTS (422).",
+        "Flags the `long-stays` feature flag (FEATURE_FLAGS env var parsed in packages/api/src/flags.js; checked at services/bookings.js line 10, giving members LONG_STAY_MAX_NIGHTS = 28). It notes the schema maximum is checked first, so members can never actually reach 28. It asks the ticket owner how the 21-night limit should interact with that flag instead of deciding alone.",
+        "Notes that services/bookings.js and web/src/bookingForm.js have no tests while pricing and format do (packages/api/test/pricing.test.js, packages/web/test/format.test.js). It warns that a new test importing services/bookings.js will throw unless DATABASE_URL is set, because packages/api/src/db.js lines 2-4 check it at import time, and suggests setting it (plus FEATURE_FLAGS if needed) before a dynamic import, using node:test and node:assert/strict as the existing tests do.",
+        "Ends with one concrete first step that can be checked (e.g. run `npm install && npm test` and see 5 passing tests, or change openapi.json, run `npm run gen` and confirm `npm run gen:check` passes and the diff shows only the nights maximum), scoped to this ticket.",
+        "Invents no commands, files, functions or settings that are not in the repo (e.g. no yarn scripts, no api/config.js, no `npm run lint` or `npm run build`, no existing bookings test, no .env file claimed to exist)."
+      ]
+    }
+  ]
+}

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

@@ -0,0 +1,23 @@
+name: ci
+on: [push, pull_request]
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    services:
+      postgres:
+        image: postgres:16
+        env:
+          POSTGRES_PASSWORD: example-only
+        ports: ["5432:5432"]
+    env:
+      DATABASE_URL: postgres://postgres:example-only@localhost:5432/postgres
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-node@v4
+        with:
+          node-version: 20
+      - run: npm ci
+      - run: npm run gen:types && git diff --exit-code src/generated
+      - run: npm run lint
+      - run: npm run typecheck
+      - run: npm test -- --coverage

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

@@ -0,0 +1,23 @@
+# shipwright-api
+
+The shipments API for the Harbourline dispatch console.
+
+## Getting started
+
+```
+npm install
+npm run dev
+```
+
+The server listens on port 3000.
+
+## Layout
+
+- `src/routes/` holds one file per resource.
+- Configuration lives in `config/default.json`.
+
+## Tests
+
+```
+npm test
+```

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