Biến rủi ro AI thành quy trình của teamTurning AI risk into team process

Từ "tôi cẩn thận" sang "team an toàn mặc định"From "I'm careful" to "safe by default"

Rủi ro nằm ở người kém cẩn thận nhất vào ngày bận nhất: dán .env vào ChatGPT cá nhân, thêm package AI gợi ý mới xuất hiện trên npm 3 ngày, duyệt 400 dòng code AI sinh mà không có test. Những việc này không cần thêm kiến thức. Chúng cần quy trình. Risk lives with the least careful person on their busiest day: pasting .env into personal ChatGPT, adding an AI-suggested package that appeared on npm 3 days ago, approving 400 lines of untested AI code. None of these needs more knowledge. They need process.

Ba lớp bù lỗ cho nhau. Thiếu lớp nào, rủi ro lọt qua đúng chỗ đó.The layers cover each other's holes. Drop one and risk slips through exactly there.

Ví dụ: cửa hàng online 5 dev, sau M2Example: a 5-dev online shop, after M2

Repo shop-web (Next.js) và shop-api (Node), triển khai bằng GitHub Actions. Team lead muốn: một văn bản ngắn để mọi người ký nhận, CI chặn lỗi phổ biến nhất, và cách để reviewer biết PR nào có code AI sinh. Repos shop-web (Next.js) and shop-api (Node), deployed with GitHub Actions. The team lead wants a short document everyone signs off on, CI that blocks the common mistakes, and a way for reviewers to know which PRs contain AI code.

AI usage policy 1–2 trangA 1–2 page AI usage policy

Policy 10 trang thì không ai đọc. 1–2 trang đọc trong 5 phút và nhớ được phần chính. Chi tiết (pattern secret, cấu hình CI) để ở file cấu hình. Một policy tốt trả lời đủ 7 câu hỏi: Nobody reads a 10-page policy. 1–2 pages take 5 minutes and stick. Details (secret patterns, CI config) live in config files. A good policy answers 7 questions:

Soạn policy của team bạnBuild your team's policy

Chọn công cụ, dữ liệu cấm và quy tắc review. Bên phải là policy đầy đủ 7 mục và PR template tương ứng. Thước đo cho biết policy còn nằm trong 1–2 trang không. Mọi thay đổi lưu trong trình duyệt của bạn. Pick tools, banned data and review rules. On the right: the full 7-section policy and the matching PR template. The meter shows whether it still fits in 1–2 pages. Everything is saved in your browser.

Đưa vào pipeline: secret + dependencyInto the pipeline: secrets + dependencies

Secret scanning có bốn tầng. Chặn càng sớm càng rẻ, nhưng chỉ tầng CI là không ai bỏ qua được.Secret scanning has four tiers. The earlier the cheaper, but only the CI tier can't be skipped.

gitleaks-action cần GITLEAKS_LICENSE cho repo của tổ chức, và bản v2 đã ngừng chạy. Chạy thẳng binary gitleaks (MIT) thì đơn giản hơn. Cảnh báo nhầm (false positive) là thứ giết chết secret scanning: đưa file test và key mẫu vào allowlist (danh sách bỏ qua) trong .gitleaks.toml, mỗi dòng ghi lý do.gitleaks-action needs a GITLEAKS_LICENSE for org repos, and v2 has stopped running. Running the gitleaks binary (MIT) directly is simpler. False positives kill secret scanning: allowlist test files and sample keys in .gitleaks.toml, each line with a reason.

import { execSync } from 'node:child_process';
const base = process.argv[2] ?? 'origin/main';
const MIN_AGE_DAYS = 30, MIN_WEEKLY = 1000;
const LICENSES = new Set(['MIT', 'ISC', 'Apache-2.0', 'BSD-2-Clause', 'BSD-3-Clause']);

const deps = (pkg) => ({ ...pkg.dependencies, ...pkg.devDependencies });
const before = deps(JSON.parse(execSync(`git show ${base}:package.json`, { encoding: 'utf8' })));
const after = deps(JSON.parse(execSync('cat package.json', { encoding: 'utf8' })));
const added = Object.keys(after).filter((n) => !(n in before));

let fail = 0;
for (const name of added) {
  const meta = await fetch(`https://registry.npmjs.org/${name}`);
  if (meta.status === 404) { console.error(`✗ ${name}: not on npm`); fail++; continue; }
  const m = await meta.json();
  const ageDays = (Date.now() - new Date(m.time.created)) / 864e5;
  const dl = await (await fetch(`https://api.npmjs.org/downloads/point/last-week/${name}`)).json();
  const lic = m.license ?? m.versions?.[m['dist-tags'].latest]?.license;
  const problems = [];
  if (ageDays < MIN_AGE_DAYS) problems.push(`${ageDays.toFixed(0)} days old`);
  if ((dl.downloads ?? 0) < MIN_WEEKLY) problems.push(`${dl.downloads ?? 0} downloads/week`);
  if (!LICENSES.has(lic)) problems.push(`licence ${lic}`);
  if (problems.length) { console.error(`✗ ${name}: ${problems.join(', ')}`); fail++; }
  else console.log(`✓ ${name}`);
}
process.exit(fail ? 1 : 0);

name: guardrails
on: pull_request
jobs:
  secrets:
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v4
        with: { fetch-depth: 0 }
      - name: gitleaks (PR diff)
        run: |
          curl -sSL https://github.com/gitleaks/gitleaks/releases/download/v8.24.0/gitleaks_8.24.0_linux_x64.tar.gz | tar xz
          ./gitleaks git --log-opts="origin/${{ github.base_ref }}..HEAD" --redact --exit-code 1
  deps:
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v4
        with: { fetch-depth: 0 }
      - uses: actions/setup-node@v4
        with: { node-version: 22 }
      - run: npm ci --ignore-scripts
      - name: known vulnerabilities
        run: npx --yes osv-scanner@latest --lockfile=package-lock.json
      - name: new packages
        run: node scripts/check-new-deps.mjs origin/${{ github.base_ref }}

Kiểm tra bản mới nhất của gitleaks và các action trước khi dùng; pin theo version hoặc commit SHA. Script báo lỗi không có nghĩa là "cấm" mà là "cần người xem": reviewer kiểm tra rồi thêm vào deps-allowlist.txt kèm lý do.Check the latest gitleaks and action versions before use; pin to a version or SHA. A failing script doesn't mean "banned" but "a human should look": the reviewer checks, then adds it to deps-allowlist.txt with a reason.

Mục "AI-assisted" trong PR templateThe "AI-assisted" PR template section

Mục này phục vụ reviewer, để biết phải nhìn kỹ chỗ nào. Nó không dùng để chấm điểm người viết. Thử tick bên dưới để xem job ai-label phản ứng thế nào.This section serves the reviewer, telling them where to look hard. It does not score the author. Tick boxes below to see how the ai-label job reacts.

Từ bản nháp đến "đã áp dụng 1 sprint"From draft to "applied for 1 sprint"

Để được duyệt, gửi team lead một trang tóm tắt, đừng gửi một bản nháp trống. Duyệt xong thì ghi tên + ngày vào cuối policy và commit vào docs/ai-usage-policy.md.To get approval, send the team lead a one-page brief, not a bare draft. Once approved, record name + date at the bottom and commit it as docs/ai-usage-policy.md.

Trước và sau M3Before and after M3

Cây quyết định 10 giâyA 10-second decision tree

Có secret, PII khách hàng
hoặc mã nguồn mật?
├─ Có  → KHÔNG. Dùng dữ liệu giả
│        hoặc mô tả bằng lời.
└─ Không
   └─ Code/dữ liệu của công ty?
      ├─ Có  → Chỉ công cụ ĐƯỢC DUYỆT
      └─ Không → Công cụ nào cũng được

Contains a secret, customer PII
or confidential source?
├─ Yes → NO. Use fake data
│        or describe it in words.
└─ No
   └─ Company code/data?
      ├─ Yes → APPROVED tools only
      └─ No  → Any tool

Code AI sinh sắp vào PR?
├─ Thêm package mới?
│    → deps-check phải qua
├─ Chạm auth/payment/admin?
│    → cần 2 approve
└─ Luôn luôn
     → có test + điền AI-assisted

AI code about to go into a PR?
├─ Adds a new package?
│    → deps-check must pass
├─ Touches auth/payment/admin?
│    → 2 approvals
└─ Always
     → tests + fill in AI-assisted

Anti-pattern

Checklist M3M3 checklist

Tự kiểm traSelf-check