skilldrop-cli 0.16.29 → 0.16.30

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

@@ -0,0 +1,38 @@
+package config
+
+import (
+	"fmt"
+	"os"
+	"strconv"
+	"time"
+)
+
+type Config struct {
+	DatabaseURL string
+	MaxAttempts int
+	RetryDelay  time.Duration
+}
+
+// FromEnv reads every setting from the environment. There is no config file.
+func FromEnv() (Config, error) {
+	cfg := Config{MaxAttempts: 5, RetryDelay: 2 * time.Second}
+	cfg.DatabaseURL = os.Getenv("TIDEPOOL_DATABASE_URL")
+	if cfg.DatabaseURL == "" {
+		return cfg, fmt.Errorf("TIDEPOOL_DATABASE_URL is required")
+	}
+	if v, ok := os.LookupEnv("TIDEPOOL_MAX_ATTEMPTS"); ok {
+		n, err := strconv.Atoi(v)
+		if err != nil {
+			return cfg, fmt.Errorf("TIDEPOOL_MAX_ATTEMPTS: %w", err)
+		}
+		cfg.MaxAttempts = n
+	}
+	if v, ok := os.LookupEnv("TIDEPOOL_RETRY_DELAY"); ok {
+		d, err := time.ParseDuration(v)
+		if err != nil {
+			return cfg, fmt.Errorf("TIDEPOOL_RETRY_DELAY: %w", err)
+		}
+		cfg.RetryDelay = d
+	}
+	return cfg, nil
+}

package/packs/dev-team/skills/codebase-tour/evals/files/3/internal/dispatch/dispatcher.go

@@ -0,0 +1,52 @@
+package dispatch
+
+import (
+	"context"
+	"fmt"
+	"time"
+
+	"example.com/tidepool/internal/config"
+	"example.com/tidepool/internal/store"
+)
+
+type Dispatcher struct {
+	cfg    config.Config
+	q      *store.Queries
+	sender Sender
+	sleep  func(time.Duration)
+}
+
+func New(cfg config.Config, q *store.Queries, s Sender) *Dispatcher {
+	return &Dispatcher{cfg: cfg, q: q, sender: s, sleep: time.Sleep}
+}
+
+// Deliver posts one event, retrying with a fixed delay until MaxAttempts.
+func (d *Dispatcher) Deliver(ctx context.Context, ev store.Event) error {
+	var last error
+	for attempt := 1; attempt <= d.cfg.MaxAttempts; attempt++ {
+		if last = d.sender.Send(ctx, ev.EndpointURL, ev.Payload); last == nil {
+			return nil
+		}
+		if attempt < d.cfg.MaxAttempts {
+			d.sleep(d.cfg.RetryDelay)
+		}
+	}
+	return fmt.Errorf("event %d: gave up after %d attempts: %w", ev.ID, d.cfg.MaxAttempts, last)
+}
+
+func (d *Dispatcher) Run(ctx context.Context) error {
+	for {
+		events, err := d.q.PendingEvents(ctx, 100)
+		if err != nil {
+			return fmt.Errorf("pending events: %w", err)
+		}
+		for _, ev := range events {
+			if err := d.Deliver(ctx, ev); err != nil {
+				_ = d.q.MarkDead(ctx, ev.ID)
+				continue
+			}
+			_ = d.q.MarkDelivered(ctx, ev.ID)
+		}
+		d.sleep(time.Second)
+	}
+}

package/packs/dev-team/skills/codebase-tour/evals/files/3/internal/dispatch/dispatcher_test.go

@@ -0,0 +1,42 @@
+package dispatch
+
+import (
+	"context"
+	"errors"
+	"testing"
+	"time"
+
+	"example.com/tidepool/internal/config"
+	"example.com/tidepool/internal/store"
+)
+
+type flakySender struct{ failures, calls int }
+
+func (f *flakySender) Send(context.Context, string, []byte) error {
+	f.calls++
+	if f.calls <= f.failures {
+		return errors.New("boom")
+	}
+	return nil
+}
+
+func TestDeliverRetriesUntilSuccess(t *testing.T) {
+	var slept []time.Duration
+	s := &flakySender{failures: 2}
+	d := &Dispatcher{cfg: config.Config{MaxAttempts: 5, RetryDelay: time.Second}, sender: s,
+		sleep: func(t time.Duration) { slept = append(slept, t) }}
+	if err := d.Deliver(context.Background(), store.Event{ID: 1}); err != nil {
+		t.Fatalf("unexpected error: %v", err)
+	}
+	if s.calls != 3 || len(slept) != 2 {
+		t.Fatalf("calls=%d slept=%v", s.calls, slept)
+	}
+}
+
+func TestDeliverGivesUpAtMaxAttempts(t *testing.T) {
+	s := &flakySender{failures: 10}
+	d := &Dispatcher{cfg: config.Config{MaxAttempts: 3}, sender: s, sleep: func(time.Duration) {}}
+	if err := d.Deliver(context.Background(), store.Event{ID: 7}); err == nil || s.calls != 3 {
+		t.Fatalf("err=%v calls=%d", err, s.calls)
+	}
+}

package/packs/dev-team/skills/codebase-tour/evals/files/3/internal/dispatch/sender.go

@@ -0,0 +1,32 @@
+package dispatch
+
+import (
+	"bytes"
+	"context"
+	"fmt"
+	"net/http"
+)
+
+type Sender interface {
+	Send(ctx context.Context, url string, body []byte) error
+}
+
+type HTTPSender struct{ client *http.Client }
+
+func NewHTTPSender(c *http.Client) *HTTPSender { return &HTTPSender{client: c} }
+
+func (s *HTTPSender) Send(ctx context.Context, url string, body []byte) error {
+	req, err := http.NewRequestWithContext(ctx, http.MethodPost, url, bytes.NewReader(body))
+	if err != nil {
+		return fmt.Errorf("build request: %w", err)
+	}
+	resp, err := s.client.Do(req)
+	if err != nil {
+		return fmt.Errorf("post %s: %w", url, err)
+	}
+	defer resp.Body.Close()
+	if resp.StatusCode >= 300 {
+		return fmt.Errorf("post %s: status %d", url, resp.StatusCode)
+	}
+	return nil
+}

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

@@ -0,0 +1,31 @@
+// Package replay re-sends dead-lettered events when an operator asks for it.
+package replay
+
+import (
+	"context"
+	"fmt"
+	"time"
+
+	"example.com/tidepool/internal/config"
+	"example.com/tidepool/internal/dispatch"
+	"example.com/tidepool/internal/store"
+)
+
+// Replay sends each dead event again. It keeps its own retry loop so an operator
+// replay never blocks on the dispatcher's queue.
+func Replay(ctx context.Context, cfg config.Config, q *store.Queries, s dispatch.Sender) error {
+	events, err := q.DeadEvents(ctx)
+	if err != nil {
+		return fmt.Errorf("dead events: %w", err)
+	}
+	for _, ev := range events {
+		for attempt := 1; attempt <= cfg.MaxAttempts; attempt++ {
+			if err := s.Send(ctx, ev.EndpointURL, ev.Payload); err == nil {
+				_ = q.MarkDelivered(ctx, ev.ID)
+				break
+			}
+			time.Sleep(cfg.RetryDelay)
+		}
+	}
+	return nil
+}

package/packs/dev-team/skills/codebase-tour/evals/files/3/internal/store/db.go

@@ -0,0 +1,35 @@
+package store
+
+import (
+	"context"
+
+	"github.com/jackc/pgx/v5/pgxpool"
+)
+
+type Queries struct{ pool *pgxpool.Pool }
+
+func Open(ctx context.Context, url string) (*pgxpool.Pool, error) { return pgxpool.New(ctx, url) }
+
+func New(pool *pgxpool.Pool) *Queries { return &Queries{pool: pool} }
+
+func (q *Queries) list(ctx context.Context, sql string, args ...any) ([]Event, error) {
+	rows, err := q.pool.Query(ctx, sql, args...)
+	if err != nil {
+		return nil, err
+	}
+	defer rows.Close()
+	var out []Event
+	for rows.Next() {
+		var e Event
+		if err := rows.Scan(&e.ID, &e.EndpointURL, &e.Payload); err != nil {
+			return nil, err
+		}
+		out = append(out, e)
+	}
+	return out, rows.Err()
+}
+
+func (q *Queries) exec(ctx context.Context, sql string, args ...any) error {
+	_, err := q.pool.Exec(ctx, sql, args...)
+	return err
+}

package/packs/dev-team/skills/codebase-tour/evals/files/3/internal/store/query.sql

@@ -0,0 +1,11 @@
+-- name: PendingEvents :many
+SELECT id, endpoint_url, payload FROM events WHERE state = 'pending' ORDER BY id LIMIT $1;
+
+-- name: DeadEvents :many
+SELECT id, endpoint_url, payload FROM events WHERE state = 'dead' ORDER BY id;
+
+-- name: MarkDelivered :exec
+UPDATE events SET state = 'delivered' WHERE id = $1;
+
+-- name: MarkDead :exec
+UPDATE events SET state = 'dead' WHERE id = $1;

package/packs/dev-team/skills/codebase-tour/evals/files/3/internal/store/query.sql.go

@@ -0,0 +1,30 @@
+// Code generated by sqlc. DO NOT EDIT.
+// versions:
+//   sqlc v1.26.0
+// source: query.sql
+
+package store
+
+import "context"
+
+type Event struct {
+	ID          int64
+	EndpointURL string
+	Payload     []byte
+}
+
+func (q *Queries) PendingEvents(ctx context.Context, limit int32) ([]Event, error) {
+	return q.list(ctx, "SELECT id, endpoint_url, payload FROM events WHERE state = 'pending' ORDER BY id LIMIT $1", limit)
+}
+
+func (q *Queries) DeadEvents(ctx context.Context) ([]Event, error) {
+	return q.list(ctx, "SELECT id, endpoint_url, payload FROM events WHERE state = 'dead' ORDER BY id")
+}
+
+func (q *Queries) MarkDelivered(ctx context.Context, id int64) error {
+	return q.exec(ctx, "UPDATE events SET state = 'delivered' WHERE id = $1", id)
+}
+
+func (q *Queries) MarkDead(ctx context.Context, id int64) error {
+	return q.exec(ctx, "UPDATE events SET state = 'dead' WHERE id = $1", id)
+}

package/packs/dev-team/skills/codebase-tour/evals/files/3/internal/store/schema.sql

@@ -0,0 +1,6 @@
+CREATE TABLE events (
+  id BIGSERIAL PRIMARY KEY,
+  endpoint_url TEXT NOT NULL,
+  payload BYTEA NOT NULL,
+  state TEXT NOT NULL DEFAULT 'pending'
+);

package/packs/dev-team/skills/codebase-tour/evals/files/3/internal/store/store_integration_test.go

@@ -0,0 +1,21 @@
+//go:build integration
+
+package store
+
+import (
+	"context"
+	"os"
+	"testing"
+)
+
+func TestMarkDeadMovesEventOutOfPending(t *testing.T) {
+	ctx := context.Background()
+	pool, err := Open(ctx, os.Getenv("TIDEPOOL_DATABASE_URL"))
+	if err != nil {
+		t.Fatal(err)
+	}
+	q := New(pool)
+	if _, err := q.PendingEvents(ctx, 1); err != nil {
+		t.Fatal(err)
+	}
+}

package/packs/dev-team/skills/codebase-tour/evals/files/3/sqlc.yaml

@@ -0,0 +1,9 @@
+version: "2"
+sql:
+  - engine: postgresql
+    queries: internal/store/query.sql
+    schema: internal/store/schema.sql
+    gen:
+      go:
+        package: store
+        out: internal/store

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

@@ -0,0 +1,4 @@
+# Required: the service refuses to start without it (see src/shipcalc/config.py)
+SHIPCALC_API_KEY=replace-me
+# Optional
+SHIPCALC_LOG_LEVEL=info

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

@@ -0,0 +1,16 @@
+name: ci
+on:
+  pull_request:
+  push:
+    branches: [main]
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-python@v5
+        with:
+          python-version: "3.12"
+      - run: make install
+      - run: make check-generated
+      - run: make test

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

@@ -0,0 +1,16 @@
+.PHONY: install run test gen check-generated
+
+install:
+	pip install -e ".[dev]"
+
+run:
+	SHIPCALC_API_KEY=dev-placeholder uvicorn shipcalc.main:app --reload --port 8080
+
+test:
+	SHIPCALC_API_KEY=test-placeholder pytest -q
+
+gen:
+	python scripts/gen_rates.py data/rates.csv src/shipcalc/generated/rates_table.py
+
+check-generated: gen
+	git diff --exit-code src/shipcalc/generated/

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"
+  }
+}