Skip to content

[osprey-ui] Features Registry page (Phase 4 of UI 2.0) — re-PR onto main (replaces #245)#276

Merged
haileyok merged 2 commits into
mainfrom
hailey/osprey-ui-2-0/phase-4-rebased
May 19, 2026
Merged

[osprey-ui] Features Registry page (Phase 4 of UI 2.0) — re-PR onto main (replaces #245)#276
haileyok merged 2 commits into
mainfrom
hailey/osprey-ui-2-0/phase-4-rebased

Conversation

@haileyok
Copy link
Copy Markdown
Member

Summary

Resubmitting — I accidentally merged #245 into the old hailey/osprey-ui-2-0/phase-3-rebased base instead of main. Same code as #245.

What this PR does

Testing

Identical content to #245's verified state — see that PR for the original test notes.

Co-authored-by: Leon Shi <101139283+cmttt@users.noreply.github.com>
Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@haileyok haileyok requested review from a team, EXBreder, ayubun and vinaysrao1 as code owners May 19, 2026 18:59
The original test_features.py SML missed several validator constraints —
bare JsonData required type annotations, `Count` isn't in upstream stdlib,
`and`/`not` need bool operands, attribute access isn't supported, the
`then=` arg to WhenRules takes effects (not features), and one test was
missing the required main.sml entry-point.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@haileyok haileyok merged commit 50f11f0 into main May 19, 2026
11 checks passed
haileyok added a commit that referenced this pull request May 19, 2026
Brings main's dependabot transitive bumps (webpack, rollup, node-forge,
yaml, jsonpath, babel), Features Registry UI (#276), GitHub Actions
hardening (#248), and Python uv bumps into the pnpm migration branch.

Conflicts resolved:
- .github/workflows/code-quality.yml: kept `pnpm install --frozen-lockfile`
  over main's `npm ci`. Also SHA-pinned `pnpm/action-setup@v6.0.0` to
  match main's actions hardening policy.
- osprey_ui/package-lock.json: kept deleted (pnpm-lock.yaml is the
  parity contract on this branch).

pnpm-lock.yaml left as-is for this commit; refreshed in the follow-up.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants