fix: accept settled ADR decisions and verify every added or edited record #102

Merged

jercik merged 1 commit from fix/adr-settlement-approved into fix/project-docs-adr-acceptance

2026-10-03 19:23:38 +00:00

jercik commented

2026-10-03 17:09:54 +00:00

Owner

Follows #98 by accepting settled decisions before implementation and verifying every added or edited ADR against its relevant evidence.

Merge this follow-up into #98 before integrating #98 with main.

Follows [#98](https://code.j4k.dev/j4k-oss/agent-skills/pulls/98) by accepting settled decisions before implementation and verifying every added or edited ADR against its relevant evidence. Merge this follow-up into #98 before integrating #98 with `main`.

jercik added 1 commit

2026-10-03 17:09:54 +00:00

fix: accept settled ADR decisions and verify every added or edited record

commit-msg / commitlint (pull_request) Successful in 23s

Details

Node tests / node:test (pull_request) Successful in 1m29s

Details

Review / Review (pull_request_target) Successful in 3m35s

Details

0d3be5ead8

forgejo-actions commented

2026-10-03 17:13:29 +00:00

Review 01M41BWWAR57FH944FRWNQX7BR — head 0d3be5ead819900087550a6ed13cb5772123222f

Review — j4k-oss/agent-skills @ `2358e488da`

Scope: diff against base tree 290d795ffdea
Status: dispatched — coverage complete (3/3 slots terminal)
Facts: current review-wide projection

Computed under:

{
  "abandonment": "abandonment-v1",
  "anchor_recipe": 1,
  "batch_policy": "batch-v1",
  "coverage": "coverage-v3",
  "dispatch_policy": "dispatch-v2",
  "grounder_version": 1,
  "grounding_read_rule": "grounding-read-v1",
  "promotion_policy": "promotion-v1",
  "report": "report-v3",
  "tally": "tally-v1",
  "triage_settle": "triage-settle-v2"
}

Findings (0)

No findings survived.

Reviewed:

general-bug (whole/default): no-claims
writing-quality (whole/default): no-claims
test-trimming (whole/default): no-claims

Other claims

grounding-pending (0)
ungrounded (0)
rejected (0)
duplicate-of (0)
unadjudicated (0)

Coverage

Coverage pass: 01M41BWWC8186Z5M1VVD7NBQBR
Accounting: complete
Slot health: healthy

lens	part	arm	unit status	runs	loss
general-bug	whole	default	no-claims	1	no
writing-quality	whole	default	no-claims	1	no
test-trimming	whole	default	no-claims	1	no

**Review** `01M41BWWAR57FH944FRWNQX7BR` — head `0d3be5ead819900087550a6ed13cb5772123222f` # Review — j4k-oss/agent-skills @ 2358e488da8a Scope: diff against base tree `290d795ffdea` Status: dispatched — coverage complete (3/3 slots terminal) Facts: current review-wide projection Computed under: ```json { "abandonment": "abandonment-v1", "anchor_recipe": 1, "batch_policy": "batch-v1", "coverage": "coverage-v3", "dispatch_policy": "dispatch-v2", "grounder_version": 1, "grounding_read_rule": "grounding-read-v1", "promotion_policy": "promotion-v1", "report": "report-v3", "tally": "tally-v1", "triage_settle": "triage-settle-v2" } ``` ## Findings (0) No findings survived. Reviewed: - general-bug (whole/default): no-claims - writing-quality (whole/default): no-claims - test-trimming (whole/default): no-claims ## Other claims - grounding-pending (0) - ungrounded (0) - rejected (0) - duplicate-of (0) - unadjudicated (0) ## Coverage Coverage pass: 01M41BWWC8186Z5M1VVD7NBQBR Accounting: complete Slot health: healthy | lens | part | arm | unit status | runs | loss | | --- | --- | --- | --- | --- | --- | | general-bug | whole | default | no-claims | 1 | no | | writing-quality | whole | default | no-claims | 1 | no | | test-trimming | whole | default | no-claims | 1 | no |

jercik merged commit 71043340d9 into fix/project-docs-adr-acceptance

2026-10-03 19:23:38 +00:00

jercik deleted branch fix/adr-settlement-approved

2026-10-03 19:23:38 +00:00

jercik referenced this pull request from a commit

2026-10-03 19:23:39 +00:00

fix: accept settled ADR decisions and verify every added or edited record (#102)

jercik referenced this pull request

2026-10-03 19:44:56 +00:00

fix(skills): record settled ADR decisions #98

jercik referenced this pull request

2026-10-03 19:45:03 +00:00