Rename package from prefab-cloud-python to reforge-python #1
Conversation
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
- Remove old submodule tests/prefab-cloud-integration-test-data - Add new submodule tests/shared-integration-test-data pointing to ReforgeHQ/integration-test-data.git - Add .idea/ and .claude/ to .gitignore 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Rename prefab_cloud_python/ to reforge_python/ directory - Rename prefab.proto to reforge.proto - Rename prefab_pb2.py to reforge_pb2.py - Rename .prefab config file to .reforge - Update pyproject.toml with new package name, URLs, and emails - Update README.md with new package imports and API key naming - Update __init__.py with new module imports and branding 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Remove all file loading methods from config_loader.py - Remove reforge_envs, reforge_config_override_dir, reforge_config_classpath_dir from options.py - Remove helper methods __construct_reforge_envs and __parse_envs - Remove unused imports (Union, glob, os, YamlParser) - Remove test config file .reforge.unit_tests.config.yaml - Simplify config_loader to only handle API-based configuration 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Rename files: client.py -> sdk.py, config_client.py -> config_sdk.py, etc. - Rename classes: Client -> ReforgeSDK, ConfigClient -> ConfigSDK, FeatureFlagClient -> FeatureFlagSDK - Rename methods: config_client() -> config_sdk(), feature_flag_client() -> feature_flag_sdk() - Rename global functions: get_client() -> get_sdk() - Rename variables: __base_client -> __base_sdk - Update imports and references throughout modules - Update README.md to use new ReforgeSDK class - Fix URL endpoints to use primary/secondary.reforge.com 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Remove logging.py and log_path_aggregator.py files - Remove logging example directories (structlogger, standard-logging) - Remove logging test files (test_logging.py, test_log_path_aggregator.py) - Remove record_log() and get_loglevel() methods from SDK - Remove logging-related options (collect_logs, collect_max_paths, bootstrap_loglevel) - Remove logging imports from main modules - Keep general telemetry functionality intact (only removed logging-specific telemetry) 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Update config_resolver.py to handle both 'prefab.current-time' and 'reforge.current-time' - Add comprehensive unit test coverage for 'reforge.current-time' functionality - Update test imports to use new reforge_python module names - Ensure both legacy and new time property names work identically 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Update test_integration.py to use new Options parameter names (sdk_key, reforge_api_urls, reforge_telemetry_url) - Update imports to use reforge_python modules and ReforgeSDK class - Remove logging import and bootstrap_loglevel parameter (no longer supported) - Update environment variable name from PREFAB_INTEGRATION_TEST_API_KEY to REFORGE_INTEGRATION_TEST_SDK_KEY - Update GitHub Actions workflow to use new environment variable name 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Rename top-level module directory to remove 'python' from the name - Update all import statements throughout the codebase - Update package configuration in pyproject.toml, ruff.toml, mypy.ini - Update README.md examples to use new module name - Update test imports to use sdk_reforge module 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Created tests/test.datafile.json with proper test configuration data - Fixed telemetry manager test to remove logging expectations - Removed sample/sample_bool references from deleted config file - Updated SDK test fixture to use datafile instead of LOCAL_ONLY - Restored all SDK test functionality with proper test data - Tests now passing: 227 (up from 162) 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Fixed options fixture to only use datafile for LOCAL_ONLY mode - This allows ALL datasource mode tests to use real API keys for cache paths - Updated feature flag SDK to use new test.datafile.json - All config SDK tests (10/10) and feature flag SDK tests (3/3) now pass 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Removed test methods that depended on sample/sample_bool config - These were referencing data from deleted .prefab.default.config.yaml 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Created sdk_reforge/VERSION file with version 0.12.0
- Updated _requests.py to load version from file instead of importlib.metadata
- Fixed HTTP header to use "sdk-python-{version}" format
- Updated log messages to say "Reforge SDK" instead of "Prefab"
- Added version update script (update_version.py) to keep VERSION and pyproject.toml in sync
- Updated pyproject.toml to include VERSION file in package
This resolves issues with weird version numbers when SDK is embedded in other projects.
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
- Fixed pre-commit config to properly exclude prefab_pb2.py - Applied black formatting to all Python files - Fixed ruff import ordering and undefined name issues - Added proper typing annotations for ReforgeSDK return type - Applied end-of-file-fixer to ensure proper line endings All core functionality preserved, just code style improvements. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Updated mypy.ini with renamed files (client.py → sdk.py, config_client.py → config_sdk.py, etc.) - Removed references to deleted logging-related files - Fixed pyproject.toml to reference prefab_pb2.py instead of reforge_pb2.py - Updated ruff.toml known-first-party packages - Added types-requests dependency for mypy type checking - Removed unused reforge.proto file (we're keeping prefab_pb2.py) - Added type: ignore for sseclient external library MyPy errors reduced from 97+ to just 1 error in 1 file. Pre-commit hooks now pass successfully! 🤖 Generated with [Claude Code](https://claude.ai/code) 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
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
Complete package rename from
prefab-cloud-pythontoreforge-pythonwith module namesdk_reforge.Major Changes
api_key→sdk_key,prefab_*→reforge_*)REFORGE_SDK_KEYwithPREFAB_API_KEYfallbackprimary.reforge.comandsecondary.reforge.comClient→ReforgeSDK,config_client()→config_sdk()reforge.current-timesupportTest Results
Breaking Changes
This is a complete package rename with breaking API changes:
from prefab_cloud_python import Client→from sdk_reforge import ReforgeSDKapi_key=→sdk_key=,prefab_api_urls=→reforge_api_urls=PREFAB_API_KEY→REFORGE_SDK_KEY(with backward compatibility)Test Plan
🤖 Generated with Claude Code