门禁不覆盖 feature matrix（no-default-features）→ feature-gating 缺陷合入后才被 CI 检出（#94 实例）

Chiusa
  • Qualcosa non va?
#14 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
3/5
Tempo stimato
1-2 giorni
Idoneità per principianti
65/100
Tipo di issue
Funzionalità
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
rust, yaml
Ambito
developer-experience, tooling

Direzione di ricerca

Start by reading the gate configuration and command handling for ~/.issue-keeper/config.yaml, then compare the recursive gate with .github/workflows/ci.yml:64-86. Add the bare cargo clippy --lib --no-default-features -- -D warnings check with the paths and timeout described in the issue. Done means the restored #94 lines are caught, normal gates pass, and the added runtime is no more than about two minutes.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

场景 gap 单（okguitar 提报）· 优先级 P2 · 依赖：无

基线 HEAD = 16fa769（jeffkit/issue-keeper）

Summary

管线本地门禁（~/.issue-keeper/config.yaml 各仓 gates）只跑 --all-features 组合，不覆盖无默认特性 / 单特性隔离构建；而 recursive 的 CI 有 feature matrix（8 变体：bare + mcp/http/cli/web_fetch/anthropic/skill-hub/web_search）。机制上：feature-gating 编译错误在本地门禁永远是盲区，只能等合入后由 CI 首红暴露。

今日实例（同型第二次）：#94（recursive a9e8a6f5）新文件 src/llm/factory.rs 在无 anthropic feature 下 clippy -D warnings 两处报错——use std::time::Duration 未使用（仅 anthropic 臂使用）、thinking_budget 参数未使用（仅转发给 anthropic 臂）。本地门禁（fmt/clippy --all-features/test）全绿放行 → CI run 37319059193 feature matrix 8 变体红 7；值守止血 9a4efb27（cfg 门控）后转绿（run 37322090026 matrix 8/8）。先例：#100 时代 clippy 未入门禁 → CI 拦截，后以"门禁补 clippy"解决——本次是同一"门禁覆盖 ≠ CI 覆盖"类的残留面。

影响

  • main CI 红只能合入后暴露：需值守止血或二次 run，成本高且打断验收节奏；
  • CI 连续红会掩盖后续真实回归（区分红因需人工读日志）；
  • 所有 Rust 仓（recursive/agentproc/im-agentproc/ilink-hub）同型档位均可能踩（机制存在但本地从未构建过该组合）。

证据（亲读）

  • ~/.issue-keeper/config.yaml（recursive 档）gates clippy：cargo clippy --workspace --all-targets --all-features -- -D warnings —— 无 --no-default-features；另一 Rust 档（同文件 115 行附近）同型。
  • recursive .github/workflows/ci.yml:64-86：feature matrix 跑 cargo check --lib --no-default-features + cargo clippy --lib --no-default-features [--features X] -- -D warnings（8 变体）。
  • 红 run 日志原文（退出 101，factory.rs:16 unused import + factory.rs:37 unused variable）。

建议

在 recursive 档 gates 增加一条 bare 检查（增量缓存下约 1 分钟，不必跑全 8 变体）：

- name: feature-matrix-bare
  command: cargo clippy --lib --no-default-features -- -D warnings
  timeout_secs: 600
  paths: ['src/**', 'Cargo.toml', 'Cargo.lock']

agentproc / im-agentproc / ilink-hub 如果有 no-default-features 构建矩阵（各自 CI 已证有），同型各加一行。

验收

合成提交（如把 #94 的 factory.rs 原始两行还原到分支）跑下发前门禁：bare 检查能报错拦截；同时正常提交门禁全绿不误报、耗时增量 ≤ ~2 分钟。

Lingua principale
Python
Stelle
0
Fork
0
Nessuna PR unita negli ultimi 30g

Preparare l'ambiente

Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di jeffkit/issue-keeper

Tutte le issue di jeffkit/issue-keeper

Issue simili

Altre issue su Python

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.