Skip to content

Add CLAUDE.md development guide for VAFT#32

Draft
HongSik-Yun-Fusion with Copilot wants to merge 1 commit into
claude/update-claude-md-X5tTifrom
copilot/sub-pr-31
Draft

Add CLAUDE.md development guide for VAFT#32
HongSik-Yun-Fusion with Copilot wants to merge 1 commit into
claude/update-claude-md-X5tTifrom
copilot/sub-pr-31

Conversation

Copilot AI commented Feb 26, 2026

Copy link
Copy Markdown
Contributor

Documents the VAFT codebase for developers: setup, testing, structure, dependencies, and a catalogued audit of known technical debt.

Contents

  • Build & Installpip install, editable dev install, pyproject.toml / setuptools notes
  • Testingpytest test/, external dependencies (HSDS server, MySQL)
  • Code Styleblack + flake8, no config files, sparse type hints
  • Project Structure — annotated directory tree covering all subpackages
  • Key Dependencies — table mapping packages to their roles
  • Database Connectivity — HSDS endpoint config, MySQL encrypted credential setup

Known Issues Catalogued

Category Location
SQL injection via f-string queries database/raw.py
Missing import re code/efit.py
Resource leaks (no try/finally) database/raw.py
Inconsistent return types database/ods.py
~374 print() instead of logging codebase-wide
Wildcard imports, no __all__ __init__.py files
Hardcoded machine constants & paths magnetics.py, raw.py, workflows
Dead/deprecated code in git test/old_codeset.py, code/efit.sav
Empty stub modules code/chease.py, code/gpec.py
Unguarded division by zero equilibrium/profile calculations
No CI, pre-commit hooks, or mocking infrastructure

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI changed the title [WIP] Add CLAUDE.md development guide for VAFT project Add CLAUDE.md development guide for VAFT Feb 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants