Skip to content

Dev#32

Merged
cute-omega merged 3 commits intomainfrom
dev
Dec 15, 2025
Merged

Dev#32
cute-omega merged 3 commits intomainfrom
dev

Conversation

@cute-omega
Copy link
Copy Markdown
Owner

fix issue #31

Copilot AI review requested due to automatic review settings December 15, 2025 20:42
@cute-omega cute-omega linked an issue Dec 15, 2025 that may be closed by this pull request
Copy link
Copy Markdown
Contributor

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 introduces JSON object sorting functionality and migrates the project to use uv for dependency management to address issue #31. The implementation adds a recursive sorting function that orders JSON keys by length (descending) then alphabetically, ensuring consistent formatting across configuration files.

Key changes:

  • Added sort_json_object() utility function to recursively sort JSON dictionaries by key length and name
  • Migrated dependency management to uv with new pyproject.toml and uv.lock files
  • Applied JSON sorting to configuration save operations in Config.save() and final config generation in main()

Reviewed changes

Copilot reviewed 6 out of 9 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
src/utils.py Added sort_json_object() function to recursively sort dictionaries by key length (desc) then alphabetically
src/Config.py Updated save() method to sort configuration before writing to file
src/main.py Applied sorting to final config output; translated TODO comment from English to Chinese
assets/final_config.json Regenerated with sorted keys according to new sorting convention
pyproject.toml New project metadata file specifying Python >=3.14 and dependencies
uv.lock New dependency lock file generated by uv for Python 3.14
requirements.txt Regenerated by uv with package hashes and dependency tree
.python-version New file specifying Python 3.14
main.py New root-level placeholder main file (appears to be auto-generated scaffolding)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@cute-omega cute-omega merged commit cc504ee into main Dec 15, 2025
7 checks passed
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.

无法访问aistudio.google.com

2 participants