Skip to content

feat(OGC-1054): make analyzer recognition profile-owned - #47

Open
pmanko wants to merge 33 commits into
codex/ogc-1054-m1-profile-lifecyclefrom
codex/ogc-1054-m2-mapping-qc
Open

feat(OGC-1054): make analyzer recognition profile-owned#47
pmanko wants to merge 33 commits into
codex/ogc-1054-m1-profile-lifecyclefrom
codex/ogc-1054-m2-mapping-qc

Conversation

@pmanko

@pmanko pmanko commented Aug 23, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

Scope

Review-ready Bridge M2 producer for profile-owned analyzer recognition and the exact contracts consumed by the shared OpenELIS Analyzer Type mapping workflow.

This PR is stacked on Bridge #46.

Implemented

  • Keep recognition rules, required protocol settings, and revision fingerprints in Bridge profiles.
  • Require explicit ASTM result-record selection and complete serial runtime settings; no hidden Java defaults.
  • Emit deterministic, plain-language recognition evidence for OpenELIS confirmation.
  • Keep analyzer protocols, parsing, control recognition, and FILE runtime in Bridge.
  • Keep OpenELIS catalog bindings and verification outside Bridge.

Exact current stack

Mock PR #43 is required by the exact-profile contract; the workflow pins that head.

Validation

  • Full Maven suite: 617 passed.
  • Exact Bridge/mock profile contract: 3 passed.
  • Live Run Tests workflow passed at the exact current head.
  • Zero unresolved review threads.

M2 remains [x] review-ready.

pmanko added 24 commits August 22, 2026 01:24
@pmanko
test(OGC-1054): define profile control recognition runtime
2f2d6a2
@pmanko
feat(OGC-1054): make control recognition profile-owned
d1033db 
Materialize recognition from the exact pinned profile revision and apply it uniformly across ASTM, HL7, CSV, Excel, and ODS parsing. Remove pushed classifier rules, operational control-lot input, and protocol-specific fallback guesses so RULES and affirmed NONE are authoritative.
@pmanko
test(OGC-1054): define recognition summary contract
4d8a064
@pmanko
feat(OGC-1054): expose control recognition summaries
364ea5c
@pmanko
test(OGC-1054): require unique profile identities
2e5c89b
@pmanko
feat(OGC-1054): reject duplicate profile identities
f584b88
@pmanko
test(OGC-1054): require recognition fingerprint summary
1365814
@pmanko
feat(OGC-1054): expose recognition fingerprints
1151250
@pmanko
test(OGC-1054): define safe recognition authoring
db905ee
@pmanko
feat(OGC-1054): project safe recognition authoring
79fa352
@pmanko
test(OGC-1054): require recognition draft endpoints
25cb812
@pmanko
test(OGC-1054): hide raw recognition fields
558f433
@pmanko
feat(OGC-1054): author recognition through profile drafts
cd8d150
@pmanko
test(OGC-1054): reject implicit control recognition
01fff7b
@pmanko
test(OGC-1054): reject invalid recognition runtime
745f735
@pmanko
feat(OGC-1054): require explicit control recognition
dd5d704
@pmanko
test(OGC-1054): expose recognition semantics
db5a40c
@pmanko
feat(OGC-1054): expose recognition semantics
30382e0
@pmanko
test(OGC-1054): require recognition catalog semantics
003b2b4
@pmanko
feat(OGC-1054): version recognition catalog semantics
64d1b89
@pmanko
test(OGC-1054): isolate ASTM listener lifecycle
30a3774
@pmanko
fix(OGC-1054): validate profile recognition patterns
3396a4b
@pmanko
refactor(OGC-1054): make ASTM result selection profile-owned
a5a5f7e
@pmanko
refactor(OGC-1054): make FILE parsing profile-owned
d826d6a
This was referenced Aug 23, 2026
Open
Open
pmanko added 4 commits August 24, 2026 20:15
@pmanko
Merge corrected OGC-1054 M1 into Bridge M2
adf250f
@pmanko
Merge corrected OGC-1054 profile contract into Bridge M2
3f1de23 
# Conflicts:
#	contracts/analyzer/v1/fixtures/analyzer-profile-astm.json
#	contracts/analyzer/v1/fixtures/analyzer-profile-file.json
#	contracts/analyzer/v1/fixtures/analyzer-profile-quantstudio.json
#	contracts/analyzer/v1/fixtures/profile-catalog-response.json
#	src/main/java/org/itech/ahb/profile/AnalyzerProfileValidator.java
#	src/test/java/org/itech/ahb/profile/AnalyzerProfileValidatorTest.java
@pmanko
ci(OGC-1054): pin corrected analyzer mock
26d6536
@pmanko
Merge corrected Bridge M1 CI into M2
445af81
pmanko added 4 commits August 26, 2026 11:41
@pmanko
Merge branch 'codex/ogc-1054-m1-profile-lifecycle' into codex/ogc-105…
f53525f 
…4-m2-mapping-qc

# Conflicts:
#	src/main/resources/analyzer-profiles/fluorocycler-xt.json
#	src/main/resources/analyzer-profiles/genexpert-astm.json
#	src/main/resources/analyzer-profiles/quantstudio.json
@pmanko
Merge branch 'codex/ogc-1054-m1-profile-lifecycle' into codex/ogc-105…
b9a9b45 
…4-m2-mapping-qc

# Conflicts:
#	src/main/resources/analyzer-profiles/fluorocycler-xt.json
#	src/main/resources/analyzer-profiles/genexpert-astm.json
#	src/main/resources/analyzer-profiles/quantstudio.json
@pmanko
fix(profiles): enforce explicit ASTM result selection
af0b8ea
@pmanko
ci(profiles): pin matching M2 mock revisions
0b498c5
@pmanko pmanko mentioned this pull request Aug 30, 2026
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@pmanko