Skip to content

feat: drop HAWE (internal repo now), publish rest of L3/L4 - #16

Merged
dborgards merged 1 commit into
mainfrom
chore/drop-hawe-package-rest-of-l4
Sep 9, 2026
Merged

feat: drop HAWE (internal repo now), publish rest of L3/L4#16
dborgards merged 1 commit into
mainfrom
chore/drop-hawe-package-rest-of-l4

Conversation

@dborgards

Copy link
Copy Markdown
Owner

Summary

  • HAWE removed entirely: CanKit.Pro.Hawe project, its tests, solution/csproj references, and every doc mention (README, arc42, SRS incl. the whole FR-HAWE-001..005 section + assumption A-6 + constraint CON-006, migration notes, release-process notes). It's moving to a separate internal repo -- even the generic SPI extension point that lived here is confidential enough not to belong in public source history.
  • Rest of L3/L4 now publishes: IsoTp, J1939Tp, CANopen, J1939, Uds drop IsPackable=false, get tracked in PublicApiSurfaceTests, and ship their first API-approval baselines. All 9 CanKit.Pro.* packages now publish from this repo.
  • Fixed in passing: SRS CON-005 said this project uses Apache-2.0 -- it's MIT.

Test plan

  • Full dotnet build -- 0 warnings, 0 errors, no Hawe references anywhere
  • Full dotnet test -- 403/403 passing
  • dotnet pack CanKit.Pro.sln produces exactly the 9 expected .nupkg/.snupkg pairs, no Hawe
  • Repo-wide grep confirms zero "Hawe" mentions outside two explanatory notes (migration-from-legacy.md, release-process.md) documenting why it's gone

…t of L3/L4

CanKit.Pro.Hawe -- the generic extension framework for a customer's
confidential private protocol -- is moving out of this public repository
into its own internal one. Removed entirely: the project, its tests,
solution/project-reference entries, and every mention across README and
docs (architecture, SRS requirements incl. the FR-HAWE-* section,
assumption A-6, constraint CON-006 -- IDs renumbered where a gap would
otherwise appear, migration and release-process notes). Nothing here
ever held more than a generic SPI extension point, but even that is
confidential enough not to belong in a public source history.

With HAWE out of the way, the remaining five L3/L4 protocol packages
(IsoTp, J1939Tp, CANopen, J1939, Uds) now publish too: drop each
project's IsPackable=false, track them in PublicApiSurfaceTests, and
check in their initial API-approval baselines. All nine CanKit.Pro.*
packages now ship from this repo; none are held back.

Also fixed while touching these docs: SRS CON-005 claimed this project
uses Apache-2.0 licensing -- it's MIT (docs/licensing.md already
explains why, correctly, elsewhere).

Verified: full build + 403/403 tests green on net10.0, `dotnet pack`
produces exactly the 9 expected packages, no remaining "Hawe" mentions
anywhere in the repo outside two explanatory notes (migration-from-
legacy.md, release-process.md) about why it's gone.
@cursor

cursor Bot commented Sep 9, 2026

Copy link
Copy Markdown

PR Summary

Medium Risk
Publishing five new public APIs locks surface via approval tests; removing HAWE is a breaking change for anyone referencing that project from this repo.

Overview
Removes CanKit.Pro.Hawe from the public repo and starts publishing the remaining L3/L4 NuGet packages.

The confidential HAWE extension framework is deleted end-to-end: solution entry, src/CanKit.Pro.Hawe, HaweFrameworkTests, and HAWE-related SRS/arc42/README content (including FR-HAWE-*, assumption A-6, and constraint CON-006). Migration and release docs now explain that HAWE continues in a separate internal repository.

Five transport/application packages are enabled for NuGet release by dropping per-project IsPackable=false on CanKit.Pro.IsoTp, J1939Tp, CANopen, J1939, and Uds. They are added to PublicApiSurfaceTests.Tracked with first-time API approval baselines, so all nine CanKit.Pro.* projects in the solution ship from dotnet pack. README and docs/release-process.md are updated to match (including target framework note net10.0 in README).

Minor doc fix: SRS CON-005 now correctly states MIT licensing instead of Apache-2.0.

Reviewed by Cursor Bugbot for commit a6d43eb. Bugbot is set up for automated code reviews on this repo. Configure here.

@dborgards
dborgards merged commit a4a937a into main Sep 9, 2026
14 of 15 checks passed
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