Skip to content

chore: remove dead scripts, tests, and stale gitignore entries#19

Merged
gdtknight merged 1 commit into
developfrom
chore/cleanup-dead-scripts
Apr 16, 2026
Merged

chore: remove dead scripts, tests, and stale gitignore entries#19
gdtknight merged 1 commit into
developfrom
chore/cleanup-dead-scripts

Conversation

@gdtknight
Copy link
Copy Markdown
Owner

Description

tests/, scripts/, .gitignore에서 수명이 다한 유지보수 아티팩트를 정리합니다. 소스 코드 변경 없이 삭제만 있는 PR이므로 빌드·렌더 동작은 영향받지 않습니다.

Related Issue

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Code refactoring
  • Performance improvement
  • Build/CI improvement

Changes

  • tests/ 디렉토리 전체 삭제test_shadow_calc.c는 테스트 대상 calculate_shadow_biasstatic으로 전환된 이후 링크 불가. test_shadow_config는 소스 없는 고아 바이너리. baselines/metadata.json은 종료된 016-compliance-refactoring 스냅샷.
  • scripts/ 죽은 스크립트 정리 (15개 파일) — 아카이브된 spec 012/013 전용 검증 스크립트, BVH 통합 수정 1회용 스크립트, 탭→스페이스 변환으로 Norm에 역행하는 fix_whitespace.sh, 존재하지 않는 경로를 참조하는 fix_line_continuations.sh, .github/scripts/sync-wiki.sh로 대체된 create_wiki.sh, macOS 절대경로가 하드코딩된 scripts/test/test_progress.sh 등.
  • .gitignore 정리 — 존재한 적 없는 테스트 바이너리 패턴(test_vector, test_parser, test_intersections, test_lighting, test_mlx)과 삭제된 test_optimizations.sh 항목 제거.
  • 유지된 파일scripts/validate_norminette.sh, scripts/test_miniRT.sh, scripts/test_scenes.sh, scripts/bench.sh. 모두 AGENTS.mdwiki/빌드-및-실행.md에서 현재 참조 중.

Testing

Test Environment

  • OS: Ubuntu (Linux 6.8.0-106-generic)
  • Compiler: cc (시스템 기본)

Test Steps

  1. make fclean && make all — 전체 재빌드 성공 확인
  2. ./miniRT scenes/valid/valid_smoke_simple.rt 구동 (시각 검증)
  3. 남은 스크립트 참조 확인: AGENTS.md, wiki/빌드-및-실행.md에서 가리키는 4개 스크립트가 실제로 존재하는지 ls scripts/로 검증

Test Results

  • All existing tests pass
  • New tests added and passing
  • Manual testing completed
  • Norminette check passed
  • No memory leaks (valgrind/leaks)

Screenshots

Checklist

  • Code follows project style guidelines (norminette)
  • Self-review of code completed
  • Comments added for complex logic
  • Documentation updated (if needed)
  • No new warnings generated
  • Tests added for new functionality
  • All tests passing locally
  • Branch is up to date with base branch
  • Commit messages follow conventional format
  • No sensitive data included

Constitution Compliance

  • 42 Norminette compliance verified
  • Code readability prioritized
  • Unit tests created and passing
  • Documentation updated (English + Korean if applicable)
  • Build verification completed
  • Git branch strategy followed
  • Logs stored in logs/ directory

Performance Impact

  • No performance impact
  • Performance improved
  • Performance decreased (justified below)

Additional Notes

  • 이 PR은 순수한 파일 삭제(977줄 deletion, 추가 0줄)로 구성되어 있어 런타임 동작에 영향이 없습니다.
  • tests/ 디렉토리가 통째로 사라지므로 향후 유닛 테스트를 재도입할 경우 새로운 빌드 타겟(예: Makefile test 규칙)과 함께 구조를 재설계하는 것이 좋습니다.

🤖 Generated with Claude Code

Clean up maintenance artifacts that have outlived their purpose:

- tests/: test_shadow_calc.c no longer links (calculate_shadow_bias
  is now static); test_shadow_config binary is an orphan with no
  source; baselines/metadata.json captured pre-refactor snapshot
  that is no longer relevant
- scripts/: remove one-off validators tied to archived specs
  (012 cleanup, 013 norminette, BVH integration fix), superseded
  helpers (create_wiki.sh replaced by .github/scripts/sync-wiki.sh),
  norm-violating tooling (fix_whitespace.sh converted tabs to spaces),
  and scripts with broken references (fix_line_continuations.sh)
- scripts/test/: all three scripts dead (hardcoded macOS path in
  test_progress.sh, outdated spec 002 validation)
- .gitignore: drop patterns for test binaries that never existed
  (test_vector, test_parser, test_mlx, etc.) and the removed
  test_optimizations.sh script

Remaining in scripts/: validate_norminette.sh, test_miniRT.sh,
test_scenes.sh, bench.sh — all referenced in AGENTS.md and wiki.

Co-Authored-By: Claude <noreply@anthropic.com>
@gdtknight gdtknight merged commit 715afc9 into develop Apr 16, 2026
16 checks passed
@gdtknight gdtknight deleted the chore/cleanup-dead-scripts branch April 16, 2026 23:25
gdtknight added a commit that referenced this pull request Apr 17, 2026
README was stuck at v2.6.0 and docs/CHANGELOG.md at v2.4.0, both
missing the accumulated release history from the package
restructuring work.

- README: rewrite the project structure tree to reflect the
  7-package feature-based layout introduced in v2.8.0 (common,
  scene, spatial, render, lighting, interact, metrics), replacing
  the stale flat layout. Extend the release history table with
  entries for v2.4.1, v2.5.0, v2.5.1, v2.6.0, v2.6.1, v2.6.2,
  v2.7.0, v2.8.0, and v2.8.1.
- CHANGELOG: add full Keep-a-Changelog entries for v2.4.1
  through v2.8.1, including the module restructurings, header
  cleanups, camera pitch fix, and the v2.8.1 maintenance
  cleanup (PRs #19, #20).

Pure documentation update — no source or build changes.

Co-Authored-By: Claude <noreply@anthropic.com>
gdtknight added a commit that referenced this pull request Apr 17, 2026
README was stuck at v2.6.0 and docs/CHANGELOG.md at v2.4.0, both
missing the accumulated release history from the package
restructuring work.

- README: rewrite the project structure tree to reflect the
  7-package feature-based layout introduced in v2.8.0 (common,
  scene, spatial, render, lighting, interact, metrics), replacing
  the stale flat layout. Extend the release history table with
  entries for v2.4.1, v2.5.0, v2.5.1, v2.6.0, v2.6.1, v2.6.2,
  v2.7.0, v2.8.0, and v2.8.1.
- CHANGELOG: add full Keep-a-Changelog entries for v2.4.1
  through v2.8.1, including the module restructurings, header
  cleanups, camera pitch fix, and the v2.8.1 maintenance
  cleanup (PRs #19, #20).

Pure documentation update — no source or build changes.

Co-authored-by: Claude <noreply@anthropic.com>
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.

1 participant