Skip to content

Conversation

@pragyan-amp
Copy link
Contributor

Summary

  • Resolved all 74 pyright type errors across the codebase
  • Fixed bug where AWS Bedrock LLM assigns semantic tags to numeric columns (LONG, BIGINT, etc.), causing Stitch jobs to fail with "Semantics on fields of type LONG are not supported"

Changes

Type fixes:

  • Aligned ModelInfo TypedDict across LLM providers
  • Fixed WizardState.models type hint
  • Added proper None handling and type narrowing throughout
  • Fixed except clause ordering and Callable imports

Stitch/PII fix:

  • Added NUMERIC_TYPES filter in stitch_tools.py to strip semantics from numeric columns
  • Updated PII detection prompt to instruct LLM not to tag numeric columns

Test plan

  • All 626 unit tests pass
  • Pyright reports 0 errors
  • Ruff linting passes

Copy link
Contributor

@punit-naik-amp punit-naik-amp left a comment

Choose a reason for hiding this comment

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

LGTM

@pragyan-amp pragyan-amp force-pushed the CHUCK-12-fix-linter-errors-in-chuck-data branch from 744fd9d to 0e09d9d Compare December 8, 2025 16:03
@pragyan-amp pragyan-amp merged commit bee191d into main Dec 8, 2025
2 checks passed
@pragyan-amp pragyan-amp deleted the CHUCK-12-fix-linter-errors-in-chuck-data branch December 8, 2025 17:03
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.

3 participants