Skip to content

feat: add automated glossary consistency validation logic#2068

Open
yunzhexu94-crypto wants to merge 2 commits into
omegat-org:masterfrom
yunzhexu94-crypto:master
Open

feat: add automated glossary consistency validation logic#2068
yunzhexu94-crypto wants to merge 2 commits into
omegat-org:masterfrom
yunzhexu94-crypto:master

Conversation

@yunzhexu94-crypto

Copy link
Copy Markdown
  • Description:
    ### Description
    This pull request introduces the core backend verification module for the automated glossary checker feature. It programmatic structures segment validation loops by cross-referencing source token streams against registered terminology dictionaries.
    
    ### What it adds:
    - Implements the foundational `GlossaryConsistencyValidator` component inside the `org.omegat.core.matching` package.
    - Safely intercepts mismatch variances where a source segment triggers a registered term but the translator's input omits the mandatory localization string.
    
    ### Compliance & Quality:
    - Adheres strictly to standard separation of concerns by keeping processing diagnostics separated from UI presentation listeners.
    - Includes safe null-pointer and blank string boundary isolation.

replit-agent and others added 2 commits June 3, 2026 21:18
Corrected `org.omegat.core.data.GlossaryEntry` to `org.omegat.gui.glossary.GlossaryEntry` in `GlossaryConsistencyValidator.java` to resolve a build error.

Replit-Commit-Author: Agent
Replit-Commit-Session-Id: 32d7b5db-6e53-4610-a925-ec0ab423319c
Replit-Commit-Checkpoint-Type: full_checkpoint
Replit-Commit-Event-Id: a349373a-fb44-4517-854f-e60b0ca637a9
Replit-Helium-Checkpoint-Created: true

@miurahr miurahr left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please don't merge unrelated files for feature enhancements; especially AI related dot folders and files.
Please add license headedr in the Java source files.
Please fix your code or changes to pass all the CI/CD checks.

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.

3 participants