Skip to content

feat: package bumps, Claude automation workflows, public site, and framework vision - #4

Merged
kolatts merged 5 commits into
mainfrom
claude/app-framework-enums-databases-e4e448
Sep 5, 2026
Merged

kolatts merged 5 commits into
mainfrom
claude/app-framework-enums-databases-e4e448

Conversation

@kolatts

@kolatts kolatts commented Sep 1, 2026

Copy link
Copy Markdown
Owner

Summary

  • Dependency bumps in Directory.Packages.props: EF Core / Extensions / ASP.NET Components 10.0.0 → 10.0.11, FluentAssertions 6.12 → 7.2.2 (held off 8.x — paid commercial license), NSubstitute 5 → 6, Test SDK 17 → 18.9, Azure SDKs / Humanizer / coverlet / xunit runner / SourceLink to latest. ApplicationInsights held at 2.23.0 (3.x is a breaking major). Build clean, 197/197 tests pass.
  • Claude automation (ported from pncli, adapted to .NET):
    • claude-review.yml — reviews every non-fork PR with framework-specific focus
    • claude-triage.yml — label an issue claude-triage and Claude implements a PR or denies with reasoning (untrusted-input hardening, Imagile Bot app token without workflows permission)
    • claude-maintenance.ymlnew scheduled workflow: weekly (Mon 13:00 UTC) NuGet bump PR with license/migration holds encoded
  • Public site: dependency-free static page in site/ + pages-deploy.yml. Hero shows the enum → seeded table transformation.
  • CLAUDE.md: Framework Vision section (enum → schema → seed → UI → verification loop), corrected package list (all six), automation docs.
  • .planning/PROPOSALS.md: prioritized improvement proposals from an imagile-app pattern review (tracked as issues).

Setup needed after merge

  • Repo settings → Pages → Source: GitHub Actions
  • Secrets/vars if not org-level: CLAUDE_CODE_OAUTH_TOKEN, IMAGILE_BOT_APP_ID, IMAGILE_BOT_PRIVATE_KEY

Test plan

  • dotnet build — 0 errors
  • dotnet test — 197 passed, 1 skipped
  • Workflow YAML validated
  • Site rendered and visually checked

🤖 Generated with Claude Code

Sunny Kolattukudy and others added 4 commits September 1, 2026 13:09
- EF Core / Extensions / ASP.NET Components 10.0.0 -> 10.0.11
- FluentAssertions 6.12.0 -> 7.2.2 (last free-license major; 8.x requires paid license)
- NSubstitute 5.3.0 -> 6.2.0, Test SDK 17.12.0 -> 18.9.0
- Azure SDKs, Humanizer, coverlet, xunit runner, SourceLink to latest
- Microsoft.ApplicationInsights held at 2.x (2.23.0); 3.x is a breaking major

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Ported from pncli and adapted to .NET:
- claude-review.yml reviews every non-fork PR with framework-specific focus
- claude-triage.yml implements or denies issues labeled claude-triage,
  with untrusted-input hardening and Imagile Bot app token (no workflows perm)
- claude-maintenance.yml runs weekly (Mon 13:00 UTC) to bump NuGet packages
  and open a PR, honoring FluentAssertions 7.x / ApplicationInsights 2.x /
  xunit 2.x license and migration holds

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Dependency-free static page (no build step) presenting the enum-to-database
vision, all six packages, and install commands. Deploys site/ to Pages on
push to main.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
- CLAUDE.md: add Framework Vision section (enum -> schema -> seed -> UI ->
  verification loop), list all six packages, document automation workflows
  and public site, genericize co-author example
- .planning/PROPOSALS.md: prioritized proposals from imagile-app pattern
  review (e2e testing package, reference-data sync extraction, convention
  rules, enum metadata caching, API integration tier)

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown

Claude encountered an error —— View job


I'll analyze this and get back to you.

…op, and auto-merge

Sync with pncli's evolved review architecture and extend it:
- claude-review.yml now reviews as github-actions[bot] so it can submit
  formal APPROVE / REQUEST_CHANGES on every PR, assesses breaking changes
  (binary/source, behavioral, dependency) and writes the verdict into the
  PR description between claude-breaking-changes markers, dispatches the
  feedback loop on REQUEST_CHANGES, and arms auto-merge (squash) on APPROVE
  for imagile-bot claude/* PRs or human PRs labeled auto-merge
- claude-pr-feedback.yml (new, ported from pncli, adapted to .NET):
  implements requested changes, pushes to the PR branch to re-fire review,
  capped at 3 pushed rounds with needs-human escalation; no-auto-fix opts a
  branch out
- auto-merge uses the Imagile Bot app token so the merge push still fires
  pages-deploy and publish-nuget
- pin claude-code-action at v1.0.213 across all workflows

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Sep 5, 2026

Copy link
Copy Markdown

Claude encountered an error —— View job


I'll analyze this and get back to you.

@kolatts
kolatts merged commit a39b6b4 into main Sep 5, 2026
1 of 2 checks passed
@kolatts
kolatts deleted the claude/app-framework-enums-databases-e4e448 branch September 5, 2026 17:33
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.

1 participant