Rename 'structure_type' to 'block_type' across models and queries#116
Merged
Conversation
…ock_type' across models and queries
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
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
Refactors reporting structure terminology from
structure_typetoblock_typeacross the entire client codebase, including models, GraphQL queries, API operations, and tests. This is a naming alignment effort to ensure consistency with the updated domain language used in the backend/API.Key Accomplishments
TaxonomyBlockStructureRequestStructureTypeenum toTaxonomyBlockStructureRequestBlockType, including the corresponding file rename (taxonomy_block_structure_request_structure_type.py→taxonomy_block_structure_request_block_type.py).structure_typefields toblock_typeacross multiple models:TaxonomyBlockStructureTaxonomyBlockStructureRequestStructureSummaryArtifactResponseChangeReportingStyleOpClassificationLiteInformationModelResponseblockTypeinstead ofstructureTypein both query variables and response field selections.op_change_reporting_styleto passblock_typeinstead ofstructure_type.LedgerClientmethods to use the newblock_typeparameter naming.RuleLiteModel: Extended theRuleLitemodel with additional fields and logic (net addition of ~45 lines), suggesting expanded functionality beyond the pure rename.block_typeterminology.Breaking Changes
structure_typefield/parameter has been renamed toblock_typeacross all public models and client methods.TaxonomyBlockStructureRequestStructureTypehas been renamed toTaxonomyBlockStructureRequestBlockType.blockType— this requires the corresponding backend API to already support this field name.Testing Notes
test_ledger_client.pyhave been updated to validate the newblock_typeterminology across query construction and response parsing.blockTypeas the field/variable name before deploying this change.Infrastructure Considerations
blockTypeinstead ofstructureType. Deploying this client change against an API still using the old terminology will result in query failures.🤖 Generated with Claude Code
Branch Info:
chore/rename-block-structuremainCo-Authored-By: Claude noreply@anthropic.com