Skip to content
Merged

v9.7.0 #2622

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
47 commits
Select commit Hold shift + click to select a range
7024647
feat: Implement %rowIndex environment variable for ViewDefinition for…
piotrszul Mar 18, 2026
9a4d45f
feat: Implement %rowIndex environment variable for ViewDefinition rep…
piotrszul Apr 1, 2026
bdbfb33
fix: Improve RowIndexCounter thread safety and suppress SonarCloud wa…
piotrszul Apr 1, 2026
ee8d008
fix: Ensure duplicate %rowIndex references in repeat produce consiste…
piotrszul Apr 7, 2026
f3d79e6
refactor: Remove obsolete RowCounter expression and related tests
piotrszul Apr 7, 2026
5086c9d
test: Add unit tests for RowIndexCounter and row counter expressions
piotrszul Apr 8, 2026
c825197
chore: Update core version to 9.7.0-SNAPSHOT
piotrszul May 11, 2026
370f90c
fix: Prevent trace duplication across FHIRPath evaluation paths
piotrszul May 18, 2026
435c20c
fix: Address review findings in trace-duplication fix
piotrszul May 18, 2026
e25a799
fix: Address multi-agent review findings in trace-duplication PR
piotrszul May 18, 2026
d006b2c
fix: Suppress SonarCloud false-positive commented-code warnings
piotrszul May 18, 2026
7b4022c
fix: Prevent trace duplication in comparators and conversion logic
piotrszul May 18, 2026
4f9468e
chore: Add scan-trace-duplicates slash command for issue #2594
piotrszul May 18, 2026
c33a0e1
fix: Extend sanitiseRow to recurse into arrays of structs
piotrszul May 5, 2026
f127fe3
refactor: Simplify sanitiseRow array branch and deduplicate test fixture
piotrszul May 5, 2026
dc550b8
test: Cover heterogeneous null patterns across array-of-struct elements
johngrimes May 10, 2026
7a26e86
chore: Suppress new Trivy CVEs and upgrade mermaid to 11.15.0
piotrszul May 18, 2026
26d2e46
fix: Prevent standalone Spark tests from destroying shared SparkContext
piotrszul May 19, 2026
006d056
fix: Clear SecurityContext after each test in BulkSubmitProviderTest
piotrszul May 19, 2026
906ef49
fix: Match staged-file URIs in $import-pnp allowlist
piotrszul May 19, 2026
ca1ce49
fix: Clear Delta cache between $import-pnp integration tests
piotrszul May 19, 2026
780db51
fix: Send auth token in poisoned-manifest exfiltration test
piotrszul May 19, 2026
aa4fdc6
fix: Include WireMock port in $bulk-submit IT allowable sources
piotrszul May 19, 2026
e1ad55b
fix: Set 60s response timeout on SqlQueryRunDeltaIT WebTestClient
piotrszul May 19, 2026
6569c8c
Merge pull request #2616 from aehrc/issue/2615
johngrimes May 19, 2026
19f563f
Update OpenSpec files
johngrimes May 17, 2026
f65698b
fix: Upgrade org.hl7.fhir.core to 6.9.7 and pin Tomcat to 10.1.55
piotrszul May 19, 2026
3251634
refactor: Reduce sanitiseRow complexity by extracting per-field helpers
johngrimes May 21, 2026
d2461e6
test: Enable shareable compliance suite in default build
piotrszul May 22, 2026
eb24d8f
test: Detect regressions in supported SoF compliance tests
piotrszul May 22, 2026
64da66b
fix: Coalesce null arrays from transformTree extractor to empty arrays
piotrszul May 21, 2026
b2a7341
fix: Emit typed empty array when repeat traversal exits encoded schema
piotrszul May 22, 2026
83f93dd
refactor: Scope safeExtractor to mapChildren where it is used
piotrszul May 22, 2026
fc34bb2
test: Add regression tests, improve docs, and archive fix-repeat-type…
piotrszul May 22, 2026
844b928
test: Guard trace single-fire invariant on non-equivalent equality
johngrimes May 22, 2026
b22ed71
Merge pull request #2608 from aehrc/issue/2594
johngrimes May 22, 2026
2be22d9
Merge pull request #2624 from aehrc/issue/2619
johngrimes May 22, 2026
4abfd46
Merge pull request #2599 from aehrc/issue/2592
johngrimes May 22, 2026
75c220c
Merge branch 'release/9.7.0' into issue/2560
johngrimes May 27, 2026
6eeb067
fix: Apply Bunsen license header to RowIndexCounter files
johngrimes May 27, 2026
4778968
fix: Pass traceCollector through SingleResourceEvaluator.withVariable
johngrimes May 28, 2026
e45e875
Merge pull request #2573 from aehrc/issue/2560
johngrimes May 28, 2026
6807965
fix: Remove invalid @throws tag from SqlFunctions.let Javadoc
johngrimes May 28, 2026
c324f9e
fix: Activate sofComplianceReport profile in pre-release workflow
johngrimes May 28, 2026
8658bd3
fix: Remove erroneous Apache Jena import from TimeCollection
johngrimes May 28, 2026
48c2533
chore: Upgrade HAPI FHIR to 8.10.0
johngrimes May 28, 2026
9f2a36d
test: Harden batch-update regression coverage and clean test harness
johngrimes May 28, 2026
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
152 changes: 152 additions & 0 deletions .claude/commands/opsx-apply.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,152 @@
---
description: Implement tasks from an OpenSpec change (Experimental)
---

Implement tasks from an OpenSpec change.

**Input**: Optionally specify a change name (e.g., `/opsx-apply add-auth`). If omitted, check if it can be inferred from conversation context. If vague or ambiguous you MUST prompt for available changes.

**Steps**

1. **Select the change**

If a name is provided, use it. Otherwise:
- Infer from conversation context if the user mentioned a change
- Auto-select if only one active change exists
- If ambiguous, run `openspec list --json` to get available changes and use the **AskUserQuestion tool** to let the user select

Always announce: "Using change: <name>" and how to override (e.g., `/opsx-apply <other>`).

2. **Check status to understand the schema**

```bash
openspec status --change "<name>" --json
```

Parse the JSON to understand:
- `schemaName`: The workflow being used (e.g., "spec-driven")
- Which artifact contains the tasks (typically "tasks" for spec-driven, check status for others)

3. **Get apply instructions**

```bash
openspec instructions apply --change "<name>" --json
```

This returns:
- `contextFiles`: artifact ID -> array of concrete file paths (varies by schema)
- Progress (total, complete, remaining)
- Task list with status
- Dynamic instruction based on current state

**Handle states:**
- If `state: "blocked"` (missing artifacts): show message, suggest using `/opsx-continue`
- If `state: "all_done"`: congratulate, suggest archive
- Otherwise: proceed to implementation

4. **Read context files**

Read every file path listed under `contextFiles` from the apply instructions output.
The files depend on the schema being used:
- **spec-driven**: proposal, specs, design, tasks
- Other schemas: follow the contextFiles from CLI output

5. **Show current progress**

Display:
- Schema being used
- Progress: "N/M tasks complete"
- Remaining tasks overview
- Dynamic instruction from CLI

6. **Implement tasks (loop until done or blocked)**

For each pending task:
- Show which task is being worked on
- Make the code changes required
- Keep changes minimal and focused
- Mark task complete in the tasks file: `- [ ]` → `- [x]`
- Continue to next task

**Pause if:**
- Task is unclear → ask for clarification
- Implementation reveals a design issue → suggest updating artifacts
- Error or blocker encountered → report and wait for guidance
- User interrupts

7. **On completion or pause, show status**

Display:
- Tasks completed this session
- Overall progress: "N/M tasks complete"
- If all done: suggest archive
- If paused: explain why and wait for guidance

**Output During Implementation**

```
## Implementing: <change-name> (schema: <schema-name>)

Working on task 3/7: <task description>
[...implementation happening...]
✓ Task complete

Working on task 4/7: <task description>
[...implementation happening...]
✓ Task complete
```

**Output On Completion**

```
## Implementation Complete

**Change:** <change-name>
**Schema:** <schema-name>
**Progress:** 7/7 tasks complete ✓

### Completed This Session
- [x] Task 1
- [x] Task 2
...

All tasks complete! You can archive this change with `/opsx-archive`.
```

**Output On Pause (Issue Encountered)**

```
## Implementation Paused

**Change:** <change-name>
**Schema:** <schema-name>
**Progress:** 4/7 tasks complete

### Issue Encountered
<description of the issue>

**Options:**
1. <option 1>
2. <option 2>
3. Other approach

What would you like to do?
```

**Guardrails**

- Keep going through tasks until done or blocked
- Always read context files before starting (from the apply instructions output)
- If task is ambiguous, pause and ask before implementing
- If implementation reveals issues, pause and suggest artifact updates
- Keep code changes minimal and scoped to each task
- Update task checkbox immediately after completing each task
- Pause on errors, blockers, or unclear requirements - don't guess
- Use contextFiles from CLI output, don't assume specific file names

**Fluid Workflow Integration**

This skill supports the "actions on a change" model:

- **Can be invoked anytime**: Before all artifacts are done (if tasks exist), after partial implementation, interleaved with other actions
- **Allows artifact updates**: If implementation reveals design issues, suggest updating artifacts - not phase-locked, work fluidly
156 changes: 156 additions & 0 deletions .claude/commands/opsx-archive.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,156 @@
---
description: Archive a completed change in the experimental workflow
---

Archive a completed change in the experimental workflow.

**Input**: Optionally specify a change name after `/opsx-archive` (e.g., `/opsx-archive add-auth`). If omitted, check if it can be inferred from conversation context. If vague or ambiguous you MUST prompt for available changes.

**Steps**

1. **If no change name provided, prompt for selection**

Run `openspec list --json` to get available changes. Use the **AskUserQuestion tool** to let the user select.

Show only active changes (not already archived).
Include the schema used for each change if available.

**IMPORTANT**: Do NOT guess or auto-select a change. Always let the user choose.

2. **Check artifact completion status**

Run `openspec status --change "<name>" --json` to check artifact completion.

Parse the JSON to understand:
- `schemaName`: The workflow being used
- `artifacts`: List of artifacts with their status (`done` or other)

**If any artifacts are not `done`:**
- Display warning listing incomplete artifacts
- Prompt user for confirmation to continue
- Proceed if user confirms

3. **Check task completion status**

Read the tasks file (typically `tasks.md`) to check for incomplete tasks.

Count tasks marked with `- [ ]` (incomplete) vs `- [x]` (complete).

**If incomplete tasks found:**
- Display warning showing count of incomplete tasks
- Prompt user for confirmation to continue
- Proceed if user confirms

**If no tasks file exists:** Proceed without task-related warning.

4. **Assess delta spec sync state**

Check for delta specs at `openspec/changes/<name>/specs/`. If none exist, proceed without sync prompt.

**If delta specs exist:**
- Compare each delta spec with its corresponding main spec at `openspec/specs/<capability>/spec.md`
- Determine what changes would be applied (adds, modifications, removals, renames)
- Show a combined summary before prompting

**Prompt options:**
- If changes needed: "Sync now (recommended)", "Archive without syncing"
- If already synced: "Archive now", "Sync anyway", "Cancel"

If user chooses sync, use Task tool (subagent_type: "general-purpose", prompt: "Use Skill tool to invoke openspec-sync-specs for change '<name>'. Delta spec analysis: <include the analyzed delta spec summary>"). Proceed to archive regardless of choice.

5. **Perform the archive**

Create the archive directory if it doesn't exist:

```bash
mkdir -p openspec/changes/archive
```

Generate target name using current date: `YYYY-MM-DD-<change-name>`

**Check if target already exists:**
- If yes: Fail with error, suggest renaming existing archive or using different date
- If no: Move the change directory to archive

```bash
mv openspec/changes/<name> openspec/changes/archive/YYYY-MM-DD-<name>
```

6. **Display summary**

Show archive completion summary including:
- Change name
- Schema that was used
- Archive location
- Spec sync status (synced / sync skipped / no delta specs)
- Note about any warnings (incomplete artifacts/tasks)

**Output On Success**

```
## Archive Complete

**Change:** <change-name>
**Schema:** <schema-name>
**Archived to:** openspec/changes/archive/YYYY-MM-DD-<name>/
**Specs:** ✓ Synced to main specs

All artifacts complete. All tasks complete.
```

**Output On Success (No Delta Specs)**

```
## Archive Complete

**Change:** <change-name>
**Schema:** <schema-name>
**Archived to:** openspec/changes/archive/YYYY-MM-DD-<name>/
**Specs:** No delta specs

All artifacts complete. All tasks complete.
```

**Output On Success With Warnings**

```
## Archive Complete (with warnings)

**Change:** <change-name>
**Schema:** <schema-name>
**Archived to:** openspec/changes/archive/YYYY-MM-DD-<name>/
**Specs:** Sync skipped (user chose to skip)

**Warnings:**
- Archived with 2 incomplete artifacts
- Archived with 3 incomplete tasks
- Delta spec sync was skipped (user chose to skip)

Review the archive if this was not intentional.
```

**Output On Error (Archive Exists)**

```
## Archive Failed

**Change:** <change-name>
**Target:** openspec/changes/archive/YYYY-MM-DD-<name>/

Target archive directory already exists.

**Options:**
1. Rename the existing archive
2. Delete the existing archive if it's a duplicate
3. Wait until a different date to archive
```

**Guardrails**

- Always prompt for change selection if not provided
- Use artifact graph (openspec status --json) for completion checking
- Don't block archive on warnings - just inform and confirm
- Preserve .openspec.yaml when moving to archive (it moves with the directory)
- Show clear summary of what happened
- If sync is requested, use the Skill tool to invoke `openspec-sync-specs` (agent-driven)
- If delta specs exist, always run the sync assessment and show the combined summary before prompting
Loading
Loading