Vibeコーディングによるリポジトリの大幅回収

[yohawing]

This pull request introduces several key changes to the YWTA Tools project, focusing on configuration management, project rules, and file organization. The most significant updates include the implementation of a robust configuration system, documentation of project rules and coding standards, and the addition of ignore rules for development-related files. Below is a structured summary of the most important changes:

Configuration System Enhancements:

• maya/ywta/config/__init__.py: Introduced the configuration system module, including BaseConfig, ConfigError, ValidationError, SettingsManager, and ConfigSchema to manage environment variables, configuration files, and default values.
• maya/ywta/config/base_config.py: Implemented the BaseConfig class and ConfigValue class to provide type-safe configuration management, validation, and support for environment variables and JSON configuration files.
• maya/ywta/config/config_schema.py: Added ConfigSchema for defining validation rules and constraints for configuration values, including type constraints, range constraints, and path validation.

Project Rules Documentation:

• .clinerules/project_summery.md: Documented project rules, coding standards, file structure, and development guidelines for Maya and Blender tools, emphasizing maintainability, PEP 8 compliance, and structured module organization.

Sprint-Specific Development Rules:

• .clinerules/current-sprint.md: Added detailed development rules for the current sprint, including tasks for improving settings management, dependency clarity, error handling, plugin architecture, and test coverage.

File Ignore Rules:

• .clineignore: Added ignore rules for compiled files, build directories, third-party libraries, IDE settings, temporary files, and large binary files to streamline version control.

[Copilot]

Pull Request Overview

This PR reorganizes and documents the YWTA Tools repository by introducing a typed configuration system, refining core module imports, and adding comprehensive project rules and ignore rules.

• Implemented a robust configuration system (config/): base classes, schema validation, defaults, and environment overrides
• Reorganized core utilities under ywta.core with new README documentation
• Added project and sprint documentation (.clinerules/) and streamlined version control with .clineignore

Reviewed Changes

Copilot reviewed 78 out of 78 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
maya/ywta/core/init.py	Centralize core utility imports (uses wildcard imports)
maya/ywta/config/config_schema.py	Add validation schema with dummy validator pattern
maya/ywta/config/base_config.py	BaseConfig implementation with inconsistent validator logic
maya/ywta/init.py	Update initialization function and imports indentation
.clineignore	Add ignore rules for build artifacts and temp files

Comments suppressed due to low confidence (1)

maya/ywta/init.py:2

• The import should be indented under def initialize():. Ensure import ywta.menu is inside the function body to match intended initialization flow and avoid top-level side effects.

import ywta.menu

[Copilot]

[nitpick] Avoid using wildcard imports in __init__.py. Explicitly import only the symbols you need or define __all__ in each submodule to keep namespace clean and improve readability.

Suggested change

	from ywta.core.maya_utils import *
	from ywta.core.node_utils import *
	from ywta.core.namespace_utils import *
	from ywta.core.ui_utils import *
	from ywta.core.settings_utils import *
	from ywta.core.geometry_utils import *
	from ywta.core.maya_utils import MayaUtilityClass, maya_function1, maya_function2
	from ywta.core.node_utils import NodeUtilityClass, node_function1, node_function2
	from ywta.core.namespace_utils import NamespaceUtilityClass, namespace_function1, namespace_function2
	from ywta.core.ui_utils import UIUtilityClass, ui_function1, ui_function2
	from ywta.core.settings_utils import SettingsUtilityClass, settings_function1, settings_function2
	from ywta.core.geometry_utils import GeometryUtilityClass, geometry_function1, geometry_function2

[Copilot]

[nitpick] The dummy validator insertion is confusing. Instead of adding and then replacing a placeholder, directly append the specific range_validator to self.validators[key] for clarity.

Suggested change

	self.add_validator(key, lambda v: True, "") # ダミー
	if key not in self.validators:
	self.validators[key] = []
	self.validators[key][-1] = range_validator
	if key not in self.validators:
	self.validators[key] = []
	self.validators[key].append(range_validator)

[Copilot]

The set method treats a None return from validator as failure. Validators currently raise exceptions rather than return booleans, so this check always fails. Wrap the call in try/except or adjust the API to expect boolean returns.

Suggested change

	if not self.validator(value):
	raise ValidationError(f"Validation failed for {self.key}: {value}")
	try:
	self.validator(value)
	except Exception as e:
	raise ValidationError(f"Validation failed for {self.key}: {value}. Error: {e}")