[Hướng dẫn 9/10] Prompt cho trợ lý lập trình AI: tạo, debug, refactor và test code

# workdays.py
from collections.abc import Iterable
from datetime import date, timedelta


def business_days_between(
    start: date, end: date, holidays: Iterable[date] = ()
) -> int:
    """Count weekdays (Mon-Fri) from start to end, inclusive, minus holidays."""
    if start > end:
        raise ValueError("start must be on or before end")

    holiday_set = set(holidays)
    count = 0
    current = start
    while current <= end:
        if current.weekday() < 5 and current not in holiday_set:
            count += 1
        current += timedelta(days=1)
    return count


# test_workdays.py
from datetime import date

import pytest

from workdays import business_days_between


def test_full_week_counts_five_days():
    assert business_days_between(date(2026, 10, 19), date(2026, 10, 25)) == 5


def test_weekday_holiday_is_skipped():
    culture_day = date(2026, 11, 3)  # thứ Ba
    assert business_days_between(date(2026, 11, 2), date(2026, 11, 8), [culture_day]) == 4


def test_start_after_end_raises():
    with pytest.raises(ValueError):
        business_days_between(date(2026, 10, 25), date(2026, 10, 19))

# (rút gọn: các test cùng ngày, chỉ cuối tuần và ngày lễ cuối tuần)

Kết quả minh họa: nội dung có thể khác nhau giữa các lần chạy và các mô hình

Nó làm gì: gom một mảng thành một object với khóa do hàm key tạo ra,
ví dụ gom các support ticket theo trạng thái.

Từng dòng một:
- <T, K extends PropertyKey>: generics. K bị giới hạn ở các kiểu được phép
  làm khóa object (string | number | symbol).
- items.reduce(..., {} as Record<K, T[]>): gộp từng phần tử vào một
  object rỗng dạng "khóa -> danh sách phần tử".
- (acc[key(item)] ||= []): phép gán logical OR (ES2021). Tạo mảng của
  nhóm nếu nó chưa tồn tại, rồi trả về mảng đó.
- .push(item); return acc: thêm phần tử và truyền object đi tiếp.

Ví dụ:
groupBy(tickets, (t) => t.status)
// {"open":[{"id":1,...},{"id":3,...}],"closed":[{"id":2,...}]}

Kết quả minh họa: nội dung có thể khác nhau giữa các lần chạy và các mô hình

1) Nguyên nhân gốc: csv.DictReader trả về mọi trường dưới dạng chuỗi, nên
   row["amount"] là "120.50", không phải số. Python không thể cộng str vào
   total kiểu int, do đó có TypeError; int() thất bại vì "120.50" không
   phải là một số nguyên.

2) Cách sửa tối thiểu: parse amount thành Decimal, giữ chính xác phần xu.

   from decimal import Decimal
   ...
   total = Decimal("0")
   ...
           total += Decimal(row["amount"])

   Kết quả: 210.40 (float sẽ lệch với các đầu vào khác: 0.1 + 0.2 là
   0.30000000000000004, sai đối với tiền)

3) Rủi ro khác: row["status"] == "paid" âm thầm bỏ qua "Paid " (hoa thường và
   khoảng trắng). Hãy cân nhắc row["status"].strip().lower().

Kết quả minh họa: nội dung có thể khác nhau giữa các lần chạy và các mô hình

Độ phức tạp: bản gốc so sánh mọi cặp, O(n²) phép so sánh chuỗi,
cộng thêm tìm kiếm tuyến tính "not in duplicates" trong danh sách. Phiên bản dưới đây là O(n).

from collections import Counter


def find_duplicate_emails(emails: list[str]) -> list[str]:
    """Return lowercased emails that appear more than once, in first-seen order."""
    counts = Counter(email.lower() for email in emails)
    seen: set[str] = set()
    result: list[str] = []
    for email in emails:
        key = email.lower()
        if counts[key] > 1 and key not in seen:
            seen.add(key)
            result.append(key)
    return result

# bench_dupes.py (rút gọn); bản gốc được giữ dưới tên find_duplicate_emails_old
emails = [f"user{random.randint(0, 3000)}@Example.com" for _ in range(5000)]
for case in (emails, [], ["[email protected]"], ["[email protected]", "[email protected]", "[email protected]", "[email protected]"]):
    assert find_duplicate_emails(case) == find_duplicate_emails_old(case)

old = timeit.timeit(lambda: find_duplicate_emails_old(emails), number=1)
new = timeit.timeit(lambda: find_duplicate_emails(emails), number=1)
print(f"old: {old:.3f}s  new: {new:.4f}s  speed-up: {old / new:.0f}x")

Kết quả minh họa: nội dung có thể khác nhau giữa các lần chạy và các mô hình

import math


def js_round(x: float) -> int:
    """Match JavaScript Math.round: halves round up (toward +infinity)."""
    return math.floor(x + 0.5)


def line_total(unit_price_cents: int, qty: int, discount_pct: float) -> int:
    gross = unit_price_cents * qty
    discount = js_round(gross * discount_pct / 100)
    return gross - discount

Các khác biệt ngữ nghĩa đã xử lý:
1. Làm tròn: Math.round làm tròn một nửa lên (2.5 -> 3); round() của Python
   làm tròn về số chẵn (2.5 -> 2), nên js_round() tái tạo lại hành vi của JS.
2. Kích thước số: số nguyên JS chỉ chính xác đến 2^53 - 1; int của Python
   không giới hạn. Kết quả giống nhau với mức giá thực tế.

Test case (unit_price_cents, qty, discount_pct -> kết quả mong đợi):
- (125, 1, 2)   -> 122   round() ngây thơ cho 123
- (1999, 3, 10) -> 5397
- (350, 1, 5)   -> 332

Kết quả minh họa: nội dung có thể khác nhau giữa các lần chạy và các mô hình

Kế hoạch
1. src/models/invoice.ts: thêm currency: string vào kiểu Invoice và
   đặt mặc định "VND" trong createInvoice().
2. src/routes/invoices.ts: mở rộng schema Zod với
   currency: z.string().regex(/^[A-Z]{3}$/).default("VND").
3. src/routes/invoices.test.ts: test rằng giá trị mặc định được áp dụng,
   "USD" được chấp nhận, còn "vnd" và "VNDX" trả về 400.

Thiếu ngữ cảnh: invoices.ts import saveInvoice từ src/db/invoices.ts,
nơi dựng câu INSERT, và bảng cần một migration. Vui lòng chia sẻ
src/db/invoices.ts và cấu trúc migration của bạn trước khi tôi viết các diff.

Kết quả minh họa: nội dung có thể khác nhau giữa các lần chạy và các mô hình

Hành vi: che mọi ký tự trừ ký tự đầu của phần local; giữ nguyên phần local
1 ký tự; chuyển domain về chữ thường; cắt khoảng trắng; từ chối địa chỉ sai định dạng.

# test_masking.py
import pytest

from masking import mask_email


@pytest.mark.parametrize(
    ("email", "expected"),
    [
        ("[email protected]", "j*******@example.com"),
        ("[email protected]", "[email protected]"),            # phần local một ký tự
        ("[email protected]", "J***@example.com"),      # domain chuyển về chữ thường
        # (rút gọn: trường hợp khoảng trắng)
    ],
)
def test_masks_valid_addresses(email, expected):
    assert mask_email(email) == expected


@pytest.mark.parametrize(
    "bad",
    ["", "no-at-sign", "@example.com", "john@", "a@[email protected]"],
)
def test_rejects_invalid_addresses(bad):
    with pytest.raises(ValueError):
        mask_email(bad)

Kết quả minh họa: nội dung có thể khác nhau giữa các lần chạy và các mô hình

| Severity | Line | Problem | Fix |
|----------|------|---------|-----|
| Critical | 5 | SQL injection: email được ghép thẳng vào câu truy vấn, nên ' OR '1'='1 trả về người dùng | Dùng tham số hóa: conn.execute("SELECT ... WHERE email = ?", (email,)) |
| High | 5-6 | SELECT * trả về mọi cột, kể cả password hash, cho client | Chỉ chọn các cột công khai (id, name, email) |
| High | 6 | dict(row) thất bại: row của sqlite3 là tuple trừ khi đặt row_factory | Đặt conn.row_factory = sqlite3.Row |
| Medium | 6 | Không có kết quả khớp trả về None, nên dict(None) raise và API trả 500 | Trả 404 khi row là None |
| Medium | 3 | Thiếu tham số email sẽ truy vấn với 'None' | Trả 400 khi không có email |
| Low | 4 | Connection không bao giờ được đóng | Dùng with closing(sqlite3.connect(...)) |

Performance: none found.

Kết quả minh họa: nội dung có thể khác nhau giữa các lần chạy và các mô hình

Pattern (dùng với re.fullmatch):
[0-9]{4}-(0[1-9]|1[0-2])-(0[1-9]|[12][0-9]|3[01])

- [0-9]{4}            năm bốn chữ số
- (0[1-9]|1[0-2])     tháng 01-12
- (0[1-9]|[12][0-9]|3[01])   ngày 01-31

Khớp:      2026-10-20, 1999-01-01, 2024-02-29, 2026-12-31
Không khớp: 2026-13-01, 2026-00-10, 2026-1-5, 2026-10-32

Không kiểm tra:
- Số ngày trong tháng: 2026-02-30 vẫn qua. Hãy dùng date.fromisoformat() cho việc đó.
- Tôi dùng [0-9], không dùng \d: trong Python, \d cũng khớp chữ số không phải ASCII
  như ２０２６ toàn chiều rộng.
- Hãy dùng re.fullmatch thay vì ^...$ với re.match: $ cũng khớp
  trước một ký tự xuống dòng ở cuối, nên "2026-10-20\n" sẽ qua.

Kết quả minh họa: nội dung có thể khác nhau giữa các lần chạy và các mô hình

find /var/log -type f -name '*.log' -size +100M -mtime -7 -exec ls -lh {} +

-type f            chỉ tệp thông thường
-name '*.log'      đặt trong dấu nháy để shell không mở rộng dấu *
-size +100M        lớn hơn 100 MiB (chạy trên find của GNU và BSD)
-mtime -7          được sửa đổi cách đây chưa đến 7 ngày
-exec ls -lh {} +  in kích thước, gom nhiều tệp vào một lần gọi ls

Kết quả minh họa: nội dung có thể khác nhau giữa các lần chạy và các mô hình

Xong.
- route.ts: rateLimit(ip, { limit: 5, window: "10 m" }) đặt trước
  bước validate; trả 429 với { success: false, error } khi vượt giới hạn.
- route.test.ts: 6 request từ một IP -> [200 x5, 429].
Kiểm tra: typecheck 0 lỗi, lint 0 cảnh báo, vitest 48 passed (1 mới).

Kết quả minh họa: nội dung có thể khác nhau giữa các lần chạy và các mô hình

1. Giả định: CSV UTF-8 có dòng tiêu đề; "." là dấu phân cách thập phân.
2. Cần kiểm tra: csv.DictReader, decimal.Decimal (stdlib, Python 3.12).
3. Dễ hỏng: "1,200.50", "Paid " có dấu cách, amount để trống.

Kết quả minh họa: nội dung có thể khác nhau giữa các lần chạy và các mô hình