skilldrop-cli 0.16.28 → 0.16.30

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

@@ -0,0 +1,23 @@
+# shipcalc
+
+Shipping quote service for the storefront. Given a cart it returns the shipping
+cost, loyalty discount and VAT.
+
+## Setup
+
+    pip install -r requirements.txt
+
+## Run locally
+
+    export FLASK_APP=shipcalc
+    flask run
+
+## Tests
+
+    python -m unittest discover
+
+## Layout
+
+- `src/shipcalc/shipping.py` - shipping cost and free-shipping rules
+- `src/shipcalc/discounts.py` - loyalty discounts
+- `src/shipcalc/tax.py` - VAT

package/packs/dev-team/skills/codebase-tour/evals/files/4/data/rates.csv

@@ -0,0 +1,4 @@
+region,currency,base_rate,per_kg,free_threshold
+EU,EUR,4.90,1.20,50.00
+UK,GBP,3.95,1.10,40.00
+US,USD,5.50,0.95,75.00

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

@@ -0,0 +1,19 @@
+[build-system]
+requires = ["setuptools>=68"]
+build-backend = "setuptools.build_meta"
+
+[project]
+name = "shipcalc"
+version = "2.3.0"
+requires-python = ">=3.11"
+dependencies = ["fastapi>=0.110", "uvicorn>=0.29"]
+
+[project.optional-dependencies]
+dev = ["pytest>=8", "httpx>=0.27"]
+
+[tool.setuptools.packages.find]
+where = ["src"]
+
+[tool.pytest.ini_options]
+testpaths = ["tests"]
+pythonpath = ["src"]

package/packs/dev-team/skills/codebase-tour/evals/files/4/scripts/gen_rates.py

@@ -0,0 +1,29 @@
+"""Generate src/shipcalc/generated/rates_table.py from data/rates.csv.
+
+Usage: python scripts/gen_rates.py data/rates.csv src/shipcalc/generated/rates_table.py
+"""
+import csv
+import sys
+
+HEADER = """# DO NOT EDIT. This file is generated by scripts/gen_rates.py from data/rates.csv.
+# Edit the CSV and run `make gen` instead. CI fails if this file is out of date.
+"""
+
+
+def main(src: str, dest: str) -> None:
+    with open(src, newline="") as fh:
+        rows = list(csv.DictReader(fh))
+    lines = [HEADER, "RATES = {"]
+    for row in rows:
+        lines.append(
+            f'    "{row["region"]}": {{"currency": "{row["currency"]}", '
+            f'"base_rate": {float(row["base_rate"])}, "per_kg": {float(row["per_kg"])}, '
+            f'"free_threshold": {float(row["free_threshold"])}}},'
+        )
+    lines.append("}")
+    with open(dest, "w") as fh:
+        fh.write("\n".join(lines) + "\n")
+
+
+if __name__ == "__main__":
+    main(sys.argv[1], sys.argv[2])

package/packs/dev-team/skills/codebase-tour/evals/files/4/src/shipcalc/api/preview.py

@@ -0,0 +1,14 @@
+"""Copy shown in the cart drawer ("Spend X more for free shipping")."""
+from decimal import Decimal
+
+# Kept here so the banner doesn't need the rates table at import time.
+FREE_SHIPPING_MIN = {"EU": Decimal("50.00"), "UK": Decimal("40.00"), "US": Decimal("75.00")}
+
+
+def free_shipping_banner(region: str, subtotal: Decimal) -> str:
+    threshold = FREE_SHIPPING_MIN.get(region)
+    if threshold is None:
+        return ""
+    if subtotal >= threshold:
+        return "You qualify for free shipping!"
+    return f"Spend {threshold - subtotal} more for free shipping."

package/packs/dev-team/skills/codebase-tour/evals/files/4/src/shipcalc/api/routes.py

@@ -0,0 +1,36 @@
+from decimal import Decimal
+
+from fastapi import APIRouter
+from pydantic import BaseModel
+
+from shipcalc.api.preview import free_shipping_banner
+from shipcalc.discounts import loyalty_discount
+from shipcalc.shipping import shipping_cost
+from shipcalc.tax import vat
+
+router = APIRouter()
+
+
+class QuoteRequest(BaseModel):
+    region: str
+    subtotal: Decimal
+    weight_kg: Decimal
+    tier: str = "bronze"
+
+
+@router.post("/quote")
+def quote(req: QuoteRequest) -> dict:
+    discount = loyalty_discount(req.tier, req.subtotal)
+    shipping = shipping_cost(req.region, req.subtotal, req.weight_kg)
+    tax = vat(req.region, req.subtotal - discount + shipping)
+    return {
+        "discount": str(discount),
+        "shipping": str(shipping),
+        "tax": str(tax),
+        "total": str(req.subtotal - discount + shipping + tax),
+    }
+
+
+@router.get("/preview/banner")
+def banner(region: str, subtotal: Decimal) -> dict:
+    return {"banner": free_shipping_banner(region, subtotal)}

package/packs/dev-team/skills/codebase-tour/evals/files/4/src/shipcalc/config.py

@@ -0,0 +1,15 @@
+import os
+from dataclasses import dataclass
+
+
+@dataclass(frozen=True)
+class Settings:
+    api_key: str
+    log_level: str
+
+
+def load_settings() -> Settings:
+    api_key = os.environ.get("SHIPCALC_API_KEY")
+    if not api_key:
+        raise RuntimeError("SHIPCALC_API_KEY must be set (see .env.example)")
+    return Settings(api_key=api_key, log_level=os.environ.get("SHIPCALC_LOG_LEVEL", "info"))

package/packs/dev-team/skills/codebase-tour/evals/files/4/src/shipcalc/discounts.py

@@ -0,0 +1,8 @@
+from decimal import Decimal
+
+LOYALTY_RATES = {"bronze": Decimal("0.00"), "silver": Decimal("0.03"), "gold": Decimal("0.05")}
+
+
+def loyalty_discount(tier: str, subtotal: Decimal) -> Decimal:
+    rate = LOYALTY_RATES.get(tier, Decimal("0.00"))
+    return (subtotal * rate).quantize(Decimal("0.01"))

package/packs/dev-team/skills/codebase-tour/evals/files/4/src/shipcalc/generated/rates_table.py

@@ -0,0 +1,8 @@
+# DO NOT EDIT. This file is generated by scripts/gen_rates.py from data/rates.csv.
+# Edit the CSV and run `make gen` instead. CI fails if this file is out of date.
+
+RATES = {
+    "EU": {"currency": "EUR", "base_rate": 4.9, "per_kg": 1.2, "free_threshold": 50.0},
+    "UK": {"currency": "GBP", "base_rate": 3.95, "per_kg": 1.1, "free_threshold": 40.0},
+    "US": {"currency": "USD", "base_rate": 5.5, "per_kg": 0.95, "free_threshold": 75.0},
+}

package/packs/dev-team/skills/codebase-tour/evals/files/4/src/shipcalc/main.py

@@ -0,0 +1,8 @@
+from fastapi import FastAPI
+
+from shipcalc.api.routes import router
+from shipcalc.config import load_settings
+
+settings = load_settings()
+app = FastAPI(title="shipcalc")
+app.include_router(router)

package/packs/dev-team/skills/codebase-tour/evals/files/4/src/shipcalc/shipping.py

@@ -0,0 +1,18 @@
+"""Shipping cost for a cart. Rates come from the generated table."""
+import os
+from decimal import Decimal, ROUND_HALF_UP
+
+from shipcalc.generated.rates_table import RATES
+
+
+def _free_shipping_enabled() -> bool:
+    # Rolled out per environment; off unless explicitly turned on.
+    return os.environ.get("SHIPCALC_FF_FREE_SHIPPING", "off") == "on"
+
+
+def shipping_cost(region: str, subtotal: Decimal, weight_kg: Decimal) -> Decimal:
+    rate = RATES[region]
+    if _free_shipping_enabled() and subtotal >= Decimal(str(rate["free_threshold"])):
+        return Decimal("0.00")
+    cost = Decimal(str(rate["base_rate"])) + Decimal(str(rate["per_kg"])) * weight_kg
+    return cost.quantize(Decimal("0.01"), rounding=ROUND_HALF_UP)

package/packs/dev-team/skills/codebase-tour/evals/files/4/src/shipcalc/tax.py

@@ -0,0 +1,7 @@
+from decimal import Decimal
+
+VAT = {"EU": Decimal("0.21"), "UK": Decimal("0.20"), "US": Decimal("0.00")}
+
+
+def vat(region: str, taxable: Decimal) -> Decimal:
+    return (taxable * VAT[region]).quantize(Decimal("0.01"))

package/packs/dev-team/skills/codebase-tour/evals/files/4/tests/conftest.py

@@ -0,0 +1,3 @@
+import os
+
+os.environ.setdefault("SHIPCALC_API_KEY", "test-placeholder")

package/packs/dev-team/skills/codebase-tour/evals/files/4/tests/unit/test_discounts.py

@@ -0,0 +1,11 @@
+from decimal import Decimal
+
+from shipcalc.discounts import loyalty_discount
+
+
+def test_gold_gets_five_percent():
+    assert loyalty_discount("gold", Decimal("100.00")) == Decimal("5.00")
+
+
+def test_unknown_tier_gets_nothing():
+    assert loyalty_discount("platinum", Decimal("100.00")) == Decimal("0.00")

package/packs/dev-team/skills/codebase-tour/evals/files/4/tests/unit/test_tax.py

@@ -0,0 +1,13 @@
+from decimal import Decimal
+
+import pytest
+
+from shipcalc.tax import vat
+
+
+@pytest.mark.parametrize(
+    "region,taxable,expected",
+    [("EU", Decimal("100.00"), Decimal("21.00")), ("UK", Decimal("100.00"), Decimal("20.00")), ("US", Decimal("100.00"), Decimal("0.00"))],
+)
+def test_vat_by_region(region, taxable, expected):
+    assert vat(region, taxable) == expected

package/packs/dev-team/skills/codebase-tour/evals/files/5/.env.example

@@ -0,0 +1,3 @@
+# Copy to .env (read by `npm run dev:api`). Placeholder values only.
+DATABASE_URL=postgres://REPLACE_ME@localhost:5432/cabinbook_dev
+PORT=3000

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

@@ -0,0 +1,13 @@
+name: CI
+on: [push, pull_request]
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-node@v4
+        with:
+          node-version: 20
+      - run: npm install --no-audit --no-fund
+      - run: npm run gen:check
+      - run: npm test

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

@@ -0,0 +1,18 @@
+# cabinbook
+
+Holiday-cabin booking: the booking API and the web booking form.
+
+## Getting started
+
+    yarn install
+    cd api && yarn start
+
+The API listens on http://localhost:3000.
+
+## Running tests
+
+    yarn test
+
+## Max stay
+
+Bookings are limited to 14 nights; the limit lives in `api/config.js`.

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

@@ -0,0 +1,13 @@
+{
+  "name": "cabinbook",
+  "private": true,
+  "workspaces": [
+    "packages/*"
+  ],
+  "scripts": {
+    "gen": "node scripts/gen-schema.mjs",
+    "gen:check": "node scripts/gen-schema.mjs --check",
+    "test": "npm test --workspaces",
+    "dev:api": "npm run dev -w @cabinbook/api"
+  }
+}

package/packs/dev-team/skills/codebase-tour/evals/files/5/packages/api/openapi.json

@@ -0,0 +1,30 @@
+{
+  "openapi": "3.0.3",
+  "info": { "title": "cabinbook", "version": "1.4.0" },
+  "paths": {
+    "/bookings": {
+      "post": {
+        "requestBody": {
+          "content": {
+            "application/json": { "schema": { "$ref": "#/components/schemas/BookingRequest" } }
+          }
+        },
+        "responses": { "201": { "description": "Created" } }
+      }
+    }
+  },
+  "components": {
+    "schemas": {
+      "BookingRequest": {
+        "type": "object",
+        "required": ["cabinId", "checkIn", "nights"],
+        "properties": {
+          "cabinId": { "type": "string" },
+          "checkIn": { "type": "string", "format": "date" },
+          "nights": { "type": "integer", "minimum": 1, "maximum": 14 },
+          "guests": { "type": "integer", "minimum": 1, "maximum": 8 }
+        }
+      }
+    }
+  }
+}

package/packs/dev-team/skills/codebase-tour/evals/files/5/packages/api/package.json

@@ -0,0 +1,15 @@
+{
+  "name": "@cabinbook/api",
+  "version": "1.4.0",
+  "private": true,
+  "type": "module",
+  "main": "src/server.js",
+  "scripts": {
+    "dev": "node --env-file=../../.env --watch src/server.js",
+    "start": "node src/server.js",
+    "test": "node --test"
+  },
+  "dependencies": {
+    "express": "^4.19.2"
+  }
+}

package/packs/dev-team/skills/codebase-tour/evals/files/5/packages/api/src/db.js

@@ -0,0 +1,19 @@
+// In production this is a pg pool built from DATABASE_URL; locally an in-memory stand-in.
+const url = process.env.DATABASE_URL;
+if (!url) {
+  throw new Error('DATABASE_URL is not set. Copy .env.example to .env.');
+}
+
+const bookings = [];
+
+export const db = {
+  url,
+  async insertBooking(row) {
+    const saved = { id: String(bookings.length + 1), ...row };
+    bookings.push(saved);
+    return saved;
+  },
+  async bookingsForCabin(cabinId) {
+    return bookings.filter((b) => b.cabinId === cabinId);
+  },
+};

package/packs/dev-team/skills/codebase-tour/evals/files/5/packages/api/src/flags.js

@@ -0,0 +1,8 @@
+// Feature flags come from a comma-separated env var, e.g. FEATURE_FLAGS=long-stays,new-pricing
+const enabled = new Set(
+  (process.env.FEATURE_FLAGS ?? '').split(',').map((f) => f.trim()).filter(Boolean),
+);
+
+export function isEnabled(flag) {
+  return enabled.has(flag);
+}

package/packs/dev-team/skills/codebase-tour/evals/files/5/packages/api/src/generated/schema.js

@@ -0,0 +1,32 @@
+// DO NOT EDIT. Generated by scripts/gen-schema.mjs from packages/api/openapi.json.
+// Change openapi.json and run `npm run gen`.
+
+export const schemas = {
+  "BookingRequest": {
+    "type": "object",
+    "required": [
+      "cabinId",
+      "checkIn",
+      "nights"
+    ],
+    "properties": {
+      "cabinId": {
+        "type": "string"
+      },
+      "checkIn": {
+        "type": "string",
+        "format": "date"
+      },
+      "nights": {
+        "type": "integer",
+        "minimum": 1,
+        "maximum": 14
+      },
+      "guests": {
+        "type": "integer",
+        "minimum": 1,
+        "maximum": 8
+      }
+    }
+  }
+};

package/packs/dev-team/skills/codebase-tour/evals/files/5/packages/api/src/middleware/validate.js

@@ -0,0 +1,20 @@
+import { schemas } from '../generated/schema.js';
+
+export function validate(schemaName) {
+  const schema = schemas[schemaName];
+  return (req, res, next) => {
+    const errors = [];
+    for (const field of schema.required ?? []) {
+      if (req.body?.[field] === undefined) errors.push(`${field} is required`);
+    }
+    for (const [field, rule] of Object.entries(schema.properties)) {
+      const value = req.body?.[field];
+      if (value === undefined) continue;
+      if (rule.type === 'integer' && !Number.isInteger(value)) errors.push(`${field} must be an integer`);
+      if (rule.minimum !== undefined && value < rule.minimum) errors.push(`${field} must be >= ${rule.minimum}`);
+      if (rule.maximum !== undefined && value > rule.maximum) errors.push(`${field} must be <= ${rule.maximum}`);
+    }
+    if (errors.length) return res.status(400).json({ errors });
+    next();
+  };
+}

package/packs/dev-team/skills/codebase-tour/evals/files/5/packages/api/src/routes/bookings.js

@@ -0,0 +1,15 @@
+import { Router } from 'express';
+import { validate } from '../middleware/validate.js';
+import { createBooking } from '../services/bookings.js';
+
+export const bookingsRouter = Router();
+
+bookingsRouter.post('/bookings', validate('BookingRequest'), async (req, res, next) => {
+  try {
+    const booking = await createBooking(req.body, req.member);
+    res.status(201).json(booking);
+  } catch (err) {
+    if (err.status) return res.status(err.status).json({ error: err.message });
+    next(err);
+  }
+});

package/packs/dev-team/skills/codebase-tour/evals/files/5/packages/api/src/server.js

@@ -0,0 +1,15 @@
+import express from 'express';
+import { bookingsRouter } from './routes/bookings.js';
+
+const app = express();
+app.use(express.json());
+app.use((req, _res, next) => {
+  // Members are identified by the upstream proxy.
+  const id = req.get('x-member-id');
+  req.member = id ? { id } : null;
+  next();
+});
+app.use(bookingsRouter);
+
+const port = Number(process.env.PORT ?? 3000);
+app.listen(port, () => console.log(`cabinbook api on :${port}`));

package/packs/dev-team/skills/codebase-tour/evals/files/5/packages/api/src/services/bookings.js

@@ -0,0 +1,23 @@
+import { db } from '../db.js';
+import { isEnabled } from '../flags.js';
+import { quote } from './pricing.js';
+
+// Business rule: no stay may be longer than two weeks.
+const MAX_NIGHTS = 14;
+const LONG_STAY_MAX_NIGHTS = 28;
+
+export function maxNightsFor(member) {
+  if (member && isEnabled('long-stays')) return LONG_STAY_MAX_NIGHTS;
+  return MAX_NIGHTS;
+}
+
+export async function createBooking({ cabinId, checkIn, nights, guests = 1 }, member) {
+  const max = maxNightsFor(member);
+  if (nights > max) {
+    const err = new Error(`Stays are limited to ${max} nights`);
+    err.status = 422;
+    throw err;
+  }
+  const price = quote({ nights, guests });
+  return db.insertBooking({ cabinId, checkIn, nights, guests, price, memberId: member?.id ?? null });
+}

package/packs/dev-team/skills/codebase-tour/evals/files/5/packages/api/src/services/pricing.js

@@ -0,0 +1,9 @@
+const NIGHTLY_RATE = 120;
+const EXTRA_GUEST_FEE = 15;
+const WEEKLY_DISCOUNT = 0.1;
+
+export function quote({ nights, guests = 1 }) {
+  const base = nights * NIGHTLY_RATE + Math.max(0, guests - 2) * EXTRA_GUEST_FEE * nights;
+  const discount = nights >= 7 ? base * WEEKLY_DISCOUNT : 0;
+  return Math.round((base - discount) * 100) / 100;
+}

package/packs/dev-team/skills/codebase-tour/evals/files/5/packages/api/test/pricing.test.js

@@ -0,0 +1,15 @@
+import { test } from 'node:test';
+import assert from 'node:assert/strict';
+import { quote } from '../src/services/pricing.js';
+
+test('charges the nightly rate for short stays', () => {
+  assert.equal(quote({ nights: 3 }), 360);
+});
+
+test('adds a fee per extra guest per night', () => {
+  assert.equal(quote({ nights: 2, guests: 4 }), 300);
+});
+
+test('applies the weekly discount from 7 nights', () => {
+  assert.equal(quote({ nights: 7 }), 756);
+});

package/packs/dev-team/skills/codebase-tour/evals/files/5/packages/web/package.json

@@ -0,0 +1,9 @@
+{
+  "name": "@cabinbook/web",
+  "version": "1.4.0",
+  "private": true,
+  "type": "module",
+  "scripts": {
+    "test": "node --test"
+  }
+}

package/packs/dev-team/skills/codebase-tour/evals/files/5/packages/web/src/bookingForm.js

@@ -0,0 +1,11 @@
+// Client-side checks for the booking form, run before POST /bookings.
+export const MAX_NIGHTS = 14;
+export const MAX_GUESTS = 8;
+
+export function validateBookingForm({ nights, guests }) {
+  const errors = {};
+  if (!Number.isInteger(nights) || nights < 1) errors.nights = 'Choose at least one night';
+  else if (nights > MAX_NIGHTS) errors.nights = `Stays are limited to ${MAX_NIGHTS} nights`;
+  if (!Number.isInteger(guests) || guests < 1 || guests > MAX_GUESTS) errors.guests = `1 to ${MAX_GUESTS} guests`;
+  return errors;
+}

package/packs/dev-team/skills/codebase-tour/evals/files/5/packages/web/src/format.js

@@ -0,0 +1,7 @@
+export function formatPrice(amount) {
+  return `€${amount.toFixed(2)}`;
+}
+
+export function formatNights(n) {
+  return n === 1 ? '1 night' : `${n} nights`;
+}

package/packs/dev-team/skills/codebase-tour/evals/files/5/packages/web/test/format.test.js

@@ -0,0 +1,12 @@
+import { test } from 'node:test';
+import assert from 'node:assert/strict';
+import { formatNights, formatPrice } from '../src/format.js';
+
+test('formats prices in euros with two decimals', () => {
+  assert.equal(formatPrice(756), '€756.00');
+});
+
+test('pluralises nights', () => {
+  assert.equal(formatNights(1), '1 night');
+  assert.equal(formatNights(14), '14 nights');
+});

package/packs/dev-team/skills/codebase-tour/evals/files/5/scripts/gen-schema.mjs

@@ -0,0 +1,27 @@
+// Generates packages/api/src/generated/schema.js from packages/api/openapi.json.
+// Usage (from the repo root): node scripts/gen-schema.mjs [--check]
+import { readFileSync, writeFileSync } from 'node:fs';
+
+const SRC = 'packages/api/openapi.json';
+const DEST = 'packages/api/src/generated/schema.js';
+
+const spec = JSON.parse(readFileSync(SRC, 'utf8'));
+const schemas = spec.components.schemas;
+
+const out = [
+  '// DO NOT EDIT. Generated by scripts/gen-schema.mjs from packages/api/openapi.json.',
+  '// Change openapi.json and run `npm run gen`.',
+  '',
+  `export const schemas = ${JSON.stringify(schemas, null, 2)};`,
+  '',
+].join('\n');
+
+if (process.argv.includes('--check')) {
+  const current = readFileSync(DEST, 'utf8');
+  if (current !== out) {
+    console.error(`${DEST} is out of date. Run \`npm run gen\`.`);
+    process.exit(1);
+  }
+} else {
+  writeFileSync(DEST, out);
+}