Move the audit-sessions epoch, positive-int and store-dir helpers into census.py

Ouverte Adaptée aux débutants
  • Quelque chose ne va pas ?
#15,539 5 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

Personne n'a encore pris cette issue.

Évaluation

Difficulté
2/5
Temps estimé
1-3 heures
Accessibilité débutants
76/100
Type d'issue
Refactorisation
Clarté
Clairement spécifiée
Activité
Active
Stack technique
python, shell
Domaine
tooling

Piste de recherche

Commence par plugins/session-flow/skills/audit-sessions/scripts/census.py, puis examine les définitions des helpers indiqués et leurs points d’appel dans collect.py, friction_scan.py, sweep.py et audit-friction/scripts/friction.py. Conserve le type argparse partagé nommé _positive afin que le texte d’erreur CLI reste identique, et préserve census.store_dir sans modification. Le travail est terminé lorsque chaque helper apparaît une seule fois dans census.py, que les scripts les utilisent, que les imports inutilisés sont supprimés et que le fragment de patch demandé est ajouté.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

Agent Brief

Type: Task
Summary: Move the audit-sessions epoch and positive-int helpers into census.py and drop the duplicate store_dir (part of restructuring pass #14802).

Current behavior:

  • _epoch is defined three times: plugins/session-flow/skills/audit-sessions/scripts/collect.py:183-190, friction_scan.py:85-92 and sweep.py:595-602.
  • The argparse type _positive is defined three times: collect.py:1010-1014, sweep.py:699-703 and plugins/session-flow/skills/audit-friction/scripts/friction.py:614-618.
  • store_dir is defined in census.py:69-70 and again in collect.py:167-168.

All four scripts already import census.

Desired behavior:
census.py holds one epoch helper and one positive-int argparse type, and the four scripts call them in place of their copies. collect.py calls census.store_dir and its own store_dir is deleted. Imports left unused are dropped.

Files:

  • plugins/session-flow/skills/audit-sessions/scripts/census.py
  • plugins/session-flow/skills/audit-sessions/scripts/collect.py
  • plugins/session-flow/skills/audit-sessions/scripts/friction_scan.py
  • plugins/session-flow/skills/audit-sessions/scripts/sweep.py
  • plugins/session-flow/skills/audit-friction/scripts/friction.py
  • a new patch fragment in .changes/session-flow/: ### Changed, "Internal restructuring, no behavior change."

Key interfaces:

  • argparse names the type function in its error for a non-integer value (invalid _positive value: 'abc'), so the shared type's __name__ stays _positive and every CLI's error text stays byte-identical.
  • census.store_dir is unchanged.

Acceptance criteria:

  • The epoch helper, the positive-int type and store_dir each exist once, in census.py, and the four scripts use them.
  • Behavior is unchanged: the same outputs, exit codes and printed results for the same inputs, including argparse errors.
  • The diff edits only the listed files and the lines of a test that assert on them when the change breaks that assertion.
  • The pull request stays within 400 changed lines and 15 hand-edited files.
  • scripts/run-ruff.sh check passes on the changed files.
  • The pytest suites in audit-sessions/scripts/tests/ and audit-friction/scripts/, and audit-friction.test.sh, pass.
  • ci-status is green.

Out of scope:

  • Other session-flow helpers, other duplication, eval cases, fixtures, and skill and agent text.
Langage dominant
Shell
Étoiles
22
Forks
2
Merge moyen
1 h 18 min
PR mergées (30 j)
975

Préparer son environnement

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de melodic-software/claude-code-plugins

Toutes les issues de melodic-software/claude-code-plugins

Issues similaires

Plus d'issues Shell/Bash

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.