cleanup(_dev): sys.modules alias to scitex_dev, drop 6,661 LOC of stale forks#257
Open
ywatanabe1989 wants to merge 1 commit intodevelopfrom
Open
cleanup(_dev): sys.modules alias to scitex_dev, drop 6,661 LOC of stale forks#257ywatanabe1989 wants to merge 1 commit intodevelopfrom
ywatanabe1989 wants to merge 1 commit intodevelopfrom
Conversation
… LOC of stale forks The umbrella's `src/scitex/_dev/` had 10 stale fork files plus _mcp/, _rename/, and _dashboard/ subpackages that diverged from the standalone scitex-dev package. scitex-dev is consistently newer (every standalone mtime ≥ every umbrella mtime) and ships the canonical implementations. Drift severity (umbrella ↔ standalone): - _rtd.py ↔ rtd.py: 4 lines (cosmetic — `._ecosystem` → `.ecosystem`) - _ssh.py ↔ ssh.py: 12, _github.py ↔ github.py: 20, _sync_remote: 26, _test ↔ test_runner: 30 - _config.py ↔ config.py: 53 (config dir moved to ~/.scitex/dev/, ECOSYSTEM is canonical) - _versions.py ↔ versions.py: 103 (additive features) - _sync.py ↔ sync.py: 135 (additive features) - _ecosystem.py ↔ ecosystem.py: 152 (added 5 packages, dropped 2 — see follow-up) - _fix.py ↔ fix.py: 472 (refactored into single-responsibility functions) Replace src/scitex/_dev/ with a 12-line sys.modules alias. Verified: - `scitex._dev is scitex_dev` → True - External callers in scitex.cli._dev_config_cli (`get_config_path`, `load_config`, `create_default_config`) still resolve via the alias. - Skill doc `_skills/general/05_version-control_02_release-automation.md` references (`sync_all`, `sync_local`, `sync_tags`, `remote_diff`, `remote_commit`, `pull_local`) all available on scitex_dev top-level. Net: +10 / -6,661 LOC. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
src/scitex/_dev/had 10 stale fork files plus_mcp/,_rename/, and_dashboard/subpackages that diverged from the standalone scitex-dev package.src/scitex/_dev/with a 12-linesys.modulesalias.Drift severity (resolved by this PR)
_rtd.pyrtd.py._ecosystem→.ecosystem)_ssh.pyssh.py_github.pygithub.py_sync_remote.pysync_remote.py_test.pytest_runner.py_config.pyconfig.py~/.scitex/dev/, ECOSYSTEM canonical)_versions.pyversions.py_sync.pysync.py_ecosystem.pyecosystem.py_fix.pyfix.pyECOSYSTEM follow-up (separate PR in scitex-dev repo)
The standalone
scitex-dev/ecosystem.pyadded 5 packages (scitex-ui, app, audio, parallel, …) but droppedscitex-notificationandscitex-pltthat the umbrella still listed. Both dropped packages exist as real GitHub repos and remain bridged via sys.modules in scitex-python — they should be re-added to ECOSYSTEM. PR will follow against ywatanabe1989/scitex-dev.Test plan
python -c "import scitex._dev as d; import scitex_dev as r; assert d is r"— passesfrom scitex._dev import get_config_path, load_config, create_default_config, sync_all, sync_local, remote_diff, remote_commit, pull_local— all resolvescitex.cli._dev_config_cli) only use top-level scitex_dev exports, unaffected