Repository navigation
feat(api): publish completed analysis-run result contract #157
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from 36 commits
Commits
Show all changes
41 commits
Select commit
Hold shift + click to select a range
a3c57a9
feat(api): publish terminal analysis result contract
seonghobae 003dae3
fix(api): preserve existing contract documentation
seonghobae 0cb693f
fix(api): restore strict terminal result implementation
seonghobae cffbf4e
test(api): cover terminal analysis result contract
seonghobae 24f00c0
feat(api): add analysis run status contract
seonghobae 30918e5
fix(api): harden analysis result serialization bindings
seonghobae 020c353
fix(api): bound accepted analysis run payloads
seonghobae f1c94f7
fix: validate terminal result bindings
seonghobae fb783f5
test(api): cover empty https origin
seonghobae 0cb9ff6
test(api): close unreachable HTTP branch
seonghobae 1fa8e9e
Merge main into completed analysis result contract
seonghobae 63a419e
fix(docs): align naruon maturity with protected main
seonghobae 7625dc6
fix(api): harden accepted receipts and provider headers
seonghobae 6852e9d
fix: harden analysis result contract boundaries
seonghobae 95add88
test: close coverage and match guarded arms
seonghobae e3770a6
test: cover provider credential header branches
seonghobae 910a54e
fix(api): reject delimiter-free credential headers
seonghobae efd5386
test: configure repository root for pytest
seonghobae 48643e5
fix(coverage): preserve multiline match guards
seonghobae 8e96984
Merge remote-tracking branch 'origin/feat/completed-analysis-result-c…
seonghobae a9a49d3
fix(coverage): respect match arm boundaries
seonghobae ef45763
fix(coverage): reject block-boundary false guards
seonghobae 3b07002
fix coverage guard after destructuring match arm
seonghobae 9238c3a
cover nested and long match guards
seonghobae e06e504
cover split nested match guard
seonghobae 12ada13
retain guards after sibling match arms
seonghobae cce90a1
feat(engine): execute cutoff-safe analysis runs (#178)
seonghobae 7402745
test(engine): include evidence available exactly at cutoff
seonghobae 3400874
fix(quality): keep sibling-block guards and lone closes in coverage gate
810e3a9
Merge current main into analysis result contract
seonghobae 88f10c4
docs: assign analysis engine ADR a unique number
seonghobae e22dcb1
fix: retain compact guarded match coverage
seonghobae 41a7218
remove dead coverage guard branch
seonghobae d245a9d
fix(coverage): stop nested match guard scans at let bindings
seonghobae 7521a15
Merge remote-tracking branch 'origin/main' into fix/pr157
seonghobae f8de47e
Merge remote-tracking branch 'origin/main' into fix/pr157b
seonghobae e65023f
merge: reconcile analysis-run result contract with protected main
seonghobae 5d1492c
Merge remote-tracking branch 'origin/main' into rebase/157
seonghobae 959108d
Merge remote-tracking branch 'origin/main' into rebase/157
seonghobae f31d822
Merge remote-tracking branch 'origin/main' into rebase/157
seonghobae f204ec9
Merge remote-tracking branch 'origin/main' into rebase/157
seonghobae File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
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
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
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
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
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
Oops, something went wrong.
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🟡 Duplicate, conflicting crate rows in topology table
The added rows for
validation_core,tepp_api, andlocation_membershipduplicate rows that already exist in the same table further down.validation_coreandtepp_apinow appear twice with different descriptions, so the table states two conflicting responsibilities for each.Was this helpful? React with 👍 or 👎 to provide feedback.