Commit Graph

2 Commits

Author SHA1 Message Date
LearningCircuit
b28c80466c refactor: cleanup remaining verified dead code across 5 areas (#3263)
* refactor: cleanup remaining verified dead code across 5 areas

Dead templates, functions, storage ABCs, eslint duplicate, dev scripts.
All verified by 40 agents (20 scanning + 20 verification).

* revert: keep 3 dev scripts that have active references

- regenerate_golden_master.py: called by pre-commit hook
  .pre-commit-hooks/check-golden-master-settings.py
- restart_server.sh: documented in API testing guide, examples,
  and multiple README files
- run_tests.py: referenced in CONTRIBUTING.md testing section

Added inline comments noting the references so future cleanup
attempts don't remove them without updating dependents.

* revert: keep restart_server_debug.sh dev script

* revert: keep debug_pytest.py and stop_server.sh dev scripts

Small utility scripts that cost nothing to keep and are useful
for developers debugging CI failures and managing the dev server.

* docs: add do-not-remove comments to all dev scripts

Each script now documents why it must be kept:
- regenerate_golden_master.py: pre-commit hook dependency
- restart_server.sh: documented in API guides and examples
- restart_server_debug.sh: companion to restart_server.sh
- run_tests.py: referenced in CONTRIBUTING.md
- debug_pytest.py: developer utility for CI failure reproduction
- stop_server.sh: companion to restart_server.sh
2026-03-28 16:03:21 +01:00
LearningCircuit
6dd481a950 Cleanup/remove temp js files (#533)
* Clean up temporary JavaScript files and reorganize test files

- Remove temp_js_*.js files that appear to be development iterations
- Move test_simple_cost.js to tests/ui_tests/ where it belongs with other UI tests
- Move package.json to tests/ directory since it's only used for test dependencies (Puppeteer)
- These changes clean up the root directory from temporary development files

* chore: Clean up root directory and organize dev scripts

- Remove redundant app.py (was calling non-existent local_deep_research.main())
- Move development utility scripts to scripts/dev/:
  - kill_servers.py: Flask server process management
  - run_tests.py: Test runner with coverage
  - debug_pytest.py: CI test debugging
- Fix kill_servers.py to use correct server command (local_deep_research.web.app)

* fix: Update UI tests workflow to use tests/package.json

The npm install command was looking for package.json in the root directory,
but it was moved to tests/ directory. Update the workflow to install npm
dependencies from the correct location.

* fix: Update full tests workflow to use tests/package.json

Another npm install command in the full tests workflow was looking for
package.json in the root directory. Update it to install from tests/
directory where the package.json is located.
2025-06-22 19:32:42 -04:00