@bananapus/ownable-v6 0.0.26 → 0.0.28

package/package.json

@@ -1,14 +1,12 @@
 {
   "name": "@bananapus/ownable-v6",
-  "version": "0.0.26",
+  "version": "0.0.28",
   "license": "MIT",
   "repository": {
     "type": "git",
     "url": "git+https://github.com/Bananapus/nana-ownable-v6"
   },
   "files": [
-    "CHANGELOG.md",
-    "foundry.lock",
     "foundry.toml",
     "references/",
     "remappings.txt",
@@ -19,8 +17,8 @@
     "node": ">=20.0.0"
   },
   "dependencies": {
-    "@bananapus/core-v6": "^0.0.54",
-    "@bananapus/permission-ids-v6": "^0.0.25",
+    "@bananapus/core-v6": "^0.0.57",
+    "@bananapus/permission-ids-v6": "^0.0.26",
     "@openzeppelin/contracts": "5.6.1"
   },
   "scripts": {

package/src/JBOwnableOverrides.sol

@@ -31,10 +31,6 @@ abstract contract JBOwnableOverrides is Context, JBPermissioned, IJBOwnable {
     /// @param projectCount The current number of minted projects.
     error JBOwnableOverrides_ProjectDoesNotExist(uint256 projectId, uint256 projectCount);
 
-    /// @notice Thrown when project-based ownership is requested without a `JBProjects` contract.
-    /// @param projectId The project owner ID that requires a non-zero `JBProjects` contract.
-    error JBOwnableOverrides_ZeroAddressProjectsWithProjectOwner(uint256 projectId);
-
     //*********************************************************************//
     // ---------------- public immutable stored properties --------------- //
     //*********************************************************************//
@@ -70,8 +66,9 @@ abstract contract JBOwnableOverrides is Context, JBPermissioned, IJBOwnable {
     /// revert until that project is created, leaving the contract unusable. Deployers must ensure that the referenced
     /// project is minted before or atomically with this contract's deployment — this is a deployment trust
     /// assumption.
-    /// @param permissions A contract storing permissions.
-    /// @param projects Mints ERC-721s that represent project ownership and transfers.
+    /// @param permissions A contract storing permissions. Assumed to be a valid deployment-time dependency.
+    /// @param projects Mints ERC-721s that represent project ownership and transfers. Assumed to be a valid
+    /// deployment-time dependency.
     /// @param initialOwner The owner if the `initialProjectIdOwner` is 0 (until ownership is transferred).
     /// @param initialProjectIdOwner The ID of the Juicebox project whose owner is this contract's owner (until
     /// ownership is transferred).
@@ -85,13 +82,6 @@ abstract contract JBOwnableOverrides is Context, JBPermissioned, IJBOwnable {
     {
         PROJECTS = projects;
 
-        // If using project-based ownership, the PROJECTS contract must be provided.
-        // Deploying with projects=address(0) and a non-zero projectId would permanently disable
-        // ownership resolution, as all ownerOf() calls would revert on the zero address.
-        if (initialProjectIdOwner != 0 && address(projects) == address(0)) {
-            revert JBOwnableOverrides_ZeroAddressProjectsWithProjectOwner({projectId: initialProjectIdOwner});
-        }
-
         // We force the inheriting contract to set an owner, as there is a low chance someone will use `JBOwnable` to
         // create an unowned contract.
         // It's more likely both were accidentally set to `0`. If you really want an unowned contract, set the owner to

package/CHANGELOG.md

@@ -1,39 +0,0 @@
-# Changelog
-
-## Scope
-
-This file describes the verified change from `nana-ownable-v5` to the current `nana-ownable-v6` repo.
-
-## Current v6 surface
-
-- `JBOwnable`
-- `JBOwnableOverrides`
-- `IJBOwnable`
-- `JBOwner`
-
-## Summary
-
-- Project-backed ownership is more defensive than in v5. The current implementation tolerates invalid or burned project NFTs by handling `ownerOf` failures instead of surfacing them as unchecked behavior.
-- The project-ownership path validates its inputs more strictly before transferring control to a project.
-- The implementation files now compile on the v6 `0.8.28` baseline instead of the old floating `^0.8.23` setup.
-
-## Verified deltas
-
-- `JBOwnable` now emits ownership-transfer events through `_msgSender()` instead of raw `msg.sender`.
-- The transfer-event path is explicitly documented to revert when the new project does not exist, instead of silently tolerating a bad target.
-- The current imports point at `core-v6`, not `core-v5`.
-
-## Breaking ABI changes
-
-- There is no meaningful public function migration in `IJBOwnable`; the interface mostly preserved its shape.
-- The important migration is behavioral: project-backed ownership resolution and transfer validation are stricter.
-
-## Indexer impact
-
-- Event names are stable, but the `caller` field is now emitted through `_msgSender()` semantics in the implementation.
-- Meta-transaction-aware consumers should prefer the v6 behavior over raw-`msg.sender` assumptions.
-
-## Migration notes
-
-- If you depended on `owner()` reverting for invalid project-backed ownership states, revisit that expectation.
-- Rebuild imports against the v6 core package and the current ownable contracts. This repo is small, but the behavior is intentionally stricter than v5.

package/foundry.lock

@@ -1,5 +0,0 @@
-{
-  "lib/forge-std": {
-    "rev": "77876f8a5b44b770a935621bb331660c90ac928e"
-  }
-}