Skip to content

Conversation

@httpdss
Copy link
Owner

@httpdss httpdss commented Oct 27, 2025

Overview

Rename the project from struct to structkit to avoid conflicts with the Python Standard Library struct module.

Changes

  • Updated package name in pyproject.toml to structkit
  • Changed CLI entry point from struct to structkit
  • Updated repository URLs to reflect new project name
  • Renamed MCP server to structkit-mcp-server
  • Updated Docker Compose service names
  • Updated all string references throughout the project

Why

The current package name struct conflicts with Python's built-in struct module, causing PyPI upload failures.

Testing

Please verify:

  • Package installs correctly: pip install structkit
  • CLI works: structkit --help
  • MCP server runs: structkit mcp

…nflict

- Update package name in pyproject.toml to 'structkit'
- Update CLI entry point from 'struct' to 'structkit'
- Update repository URLs to point to structkit
- Update MCP server name to 'structkit-mcp-server'
- Update Docker Compose service names
- Update all string references throughout project
@httpdss httpdss requested a review from Copilot October 27, 2025 15:56
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR renames the project from struct to structkit to resolve a naming conflict with Python's built-in struct module that was preventing PyPI uploads.

Key Changes:

  • Updated package name and CLI entry point from struct to structkit
  • Renamed MCP server from struct-mcp-server to structkit-mcp-server
  • Updated all repository URLs and documentation references

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
pyproject.toml Updated package name, CLI entry point, and repository URLs to structkit
struct_module/main.py Changed CLI program name to structkit
struct_module/mcp_server.py Renamed MCP server to structkit-mcp-server
docs/mcp-integration.md Updated all command references and examples to use structkit
docker-compose.yaml Renamed Docker Compose services to use structkit prefix

- Rename struct_module folder to structkit using git mv
- Update all imports and references throughout project
- Update pyproject.toml package configuration
- Update Docker Compose entry points
- Update all documentation and config files
- Update mcp_server.py to check for both 'struct' (original config key) and 'structkit' (new package name)
- Maintains backwards compatibility with existing YAML configurations
- Fixes failing test: test_mcp_get_structure_info_rich_rendering
@httpdss httpdss added the major label Oct 27, 2025
@codecov
Copy link

codecov bot commented Oct 27, 2025

Codecov Report

❌ Patch coverage is 68.88889% with 14 lines in your changes missing coverage. Please review.
✅ Project coverage is 68.72%. Comparing base (70173c5) to head (cf69133).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
structkit/main.py 0.00% 11 Missing ⚠️
structkit/commands/import.py 0.00% 1 Missing ⚠️
structkit/commands/info.py 50.00% 1 Missing ⚠️
structkit/commands/list.py 75.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #121   +/-   ##
=======================================
  Coverage   68.72%   68.72%           
=======================================
  Files          20       20           
  Lines        1554     1554           
  Branches      279      279           
=======================================
  Hits         1068     1068           
  Misses        411      411           
  Partials       75       75           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

- Replace all 'STRUCT' text references with 'StructKit' in documentation
- Update CLI command examples from 'struct' to 'structkit'
- Rename environment variables:
  - STRUCT_LOG_LEVEL → STRUCTKIT_LOG_LEVEL
  - STRUCT_HTTP_TIMEOUT → STRUCTKIT_HTTP_TIMEOUT
  - STRUCT_HTTP_RETRIES → STRUCTKIT_HTTP_RETRIES
  - STRUCT_DENY_NETWORK → STRUCTKIT_DENY_NETWORK
  - STRUCT_STRUCTURES_PATH → STRUCTKIT_STRUCTURES_PATH
- Rename constants: BASIC_STRUCT_YAML → BASIC_STRUCTKIT_YAML
- Update all documentation files, README, and code comments
- All 114 tests passing
- Add blank lines after headings (H3, H4) before lists
- Change code block language from bare backticks to 'text'
- Ensure proper spacing between headings and content
- Fixes markdown formatting for better readability and linter compliance
- Update devcontainer configuration
- Update GitHub workflows and instructions
- Update all documentation files with StructKit references
- Update website assets and styling
- Update vhs demo scripts
- Complete branding update across all materials
@httpdss httpdss merged commit 202784f into main Oct 27, 2025
5 checks passed
@httpdss httpdss deleted the feature/rename-to-structkit branch October 27, 2025 23:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants