fixed cycle deps

Author: bennobuilder

.gitignore

@@ -13,7 +13,6 @@ yarn-debug.log*
 yarn-error.log*
 .eslintcache
 package-lock.json
-yarn.lock
 
 # Editor directories and files
 .idea

examples/react/develop/fields/src/App.js

@@ -52,7 +52,8 @@ export default function App() {
             id: FIELDS.size,
             name: `Field #${FIELDS.size + 1}`,
           });
-        }}>
+        }}
+      >
         Add Field
       </button>
     </div>

examples/react/develop/simple-todo-list/src/App.js

@@ -30,7 +30,8 @@ const App = () => {
             method: 'unshift', // to add todo at the beginning of the Collection
           });
           setCurrentInput('');
-        }}>
+        }}
+      >
         Add
       </button>
       {todos.map((value) => (
@@ -41,7 +42,8 @@ const App = () => {
             onClick={() => {
               // Remove Todo at specific primary Key
               TODOS.remove(value.id).everywhere();
-            }}>
+            }}
+          >
             Remove
           </button>
         </div>

package.json

@@ -41,7 +41,8 @@
     "pack:react": "cd packages/react && yarn run prepare && yarn run pack",
     "pack:multieditor": "cd packages/multieditor && yarn run prepare && yarn run pack",
     "pack:api": "cd packages/api && yarn run prepare && yarn run pack",
-    "pack:event": "cd packages/event && yarn run prepare && yarn run pack"
+    "pack:event": "cd packages/event && yarn run prepare && yarn run pack",
+    "install:clean": "shx rm -rf yarn.lock && shx rm -rf node_modules && yarn install && lerna run install:clean"
   },
   "repository": {
     "type": "git",

packages/core/src/collection/collection.persistent.ts

@@ -1,7 +1,7 @@
 import { defineConfig } from '@agile-ts/utils';
 import { logCodeManager } from '../logCodeManager';
 import { Collection, CollectionKey, DefaultItem, ItemKey } from './collection';
-import { Group, GroupKey } from './group';
+import type { Group, GroupKey } from './group';
 import {
   CreatePersistentConfigInterface,
   getSharedStorageManager,
@@ -137,9 +137,8 @@ export class CollectionPersistent<
         // that it was loaded completely.
         // After a successful loading assign the now valid Item to the Collection.
         else {
-          const placeholderItem = this.collection().getItemWithReference(
-            itemKey
-          );
+          const placeholderItem =
+            this.collection().getItemWithReference(itemKey);
           placeholderItem?.persist({
             key: itemStorageKey,
             loadValue: false,
@@ -148,7 +147,8 @@ export class CollectionPersistent<
             followCollectionPersistKeyPattern: false, // Because of the dynamic 'storageItemKey', the key is already formatted above
           });
           if (placeholderItem?.persistent?.ready) {
-            const loadedPersistedValueIntoItem = await placeholderItem.persistent.loadPersistedValue();
+            const loadedPersistedValueIntoItem =
+              await placeholderItem.persistent.loadPersistedValue();
 
             // If successfully loaded Item value, assign Item to Collection
             if (loadedPersistedValueIntoItem) {

packages/core/src/collection/collection.ts

@@ -7,9 +7,9 @@ import {
   normalizeArray,
 } from '@agile-ts/utils';
 import { logCodeManager } from '../logCodeManager';
-import { Agile } from '../agile';
+import type { Agile } from '../agile';
 import { PatchOptionConfigInterface } from '../state';
-import { ComputedTracker } from '../computed';
+import { ComputedTracker } from '../computed/computed.tracker'; // Not imported directly from '../computed' due circular dependencies
 import { Item } from './item';
 import { SelectorConfigInterface, Selector, SelectorKey } from './selector';
 import {
@@ -1181,9 +1181,7 @@ export class Collection<DataType extends DefaultItem = DefaultItem> {
    * @public
    * @param itemKeys - Item/s with identifier/s to be removed.
    */
-  public remove(
-    itemKeys: ItemKey | Array<ItemKey>
-  ): {
+  public remove(itemKeys: ItemKey | Array<ItemKey>): {
     fromGroups: (groups: Array<ItemKey> | ItemKey) => Collection<DataType>;
     everywhere: (config?: RemoveItemsConfigInterface) => Collection<DataType>;
   } {
@@ -1518,13 +1516,12 @@ export interface CreateCollectionConfigImpl<
   initialData?: Array<DataType>;
 }
 
-export type CreateCollectionConfig<
-  DataType extends DefaultItem = DefaultItem
-> =
-  | CreateCollectionConfigImpl<DataType>
-  | ((
-      collection: Collection<DataType>
-    ) => CreateCollectionConfigImpl<DataType>);
+export type CreateCollectionConfig<DataType extends DefaultItem = DefaultItem> =
+
+    | CreateCollectionConfigImpl<DataType>
+    | ((
+        collection: Collection<DataType>
+      ) => CreateCollectionConfigImpl<DataType>);
 
 export interface CollectionConfigInterface {
   /**
@@ -1542,8 +1539,9 @@ export interface CollectionConfigInterface {
   defaultGroupKey: ItemKey;
 }
 
-export interface CollectConfigInterface<DataType = any>
-  extends AssignDataConfigInterface {
+export interface CollectConfigInterface<
+  DataType extends DefaultItem = DefaultItem
+> extends AssignDataConfigInterface {
   /**
    * In which way the collected data should be added to the Collection.
    * - 'push' =  at the end

packages/core/src/collection/group/group.observer.ts

@@ -6,7 +6,7 @@ import {
   Observer,
   RuntimeJob,
 } from '../../runtime';
-import { Group } from './index';
+import type { Group } from './index';
 import { DefaultItem } from '../collection';
 import { logCodeManager } from '../../logCodeManager';
 

packages/core/src/collection/group/index.ts

@@ -7,10 +7,10 @@ import {
   StateObserver,
   StateObserversInterface,
 } from '../../state';
-import { Collection, DefaultItem, ItemKey } from '../collection';
+import type { Collection, DefaultItem, ItemKey } from '../collection';
 import { GroupIngestConfigInterface, GroupObserver } from './group.observer';
-import { ComputedTracker } from '../../computed';
-import { Item } from '../item';
+import { ComputedTracker } from '../../computed/computed.tracker'; // Not imported directly from '../computed' due circular dependencies
+import type { Item } from '../item';
 import { CollectionPersistent } from '../collection.persistent';
 
 export class Group<
@@ -432,8 +432,10 @@ export class Group<
 
 export type GroupKey = string | number;
 
-export interface GroupObservers<ValueType = any, DataType = any>
-  extends StateObserversInterface<ValueType> {
+export interface GroupObservers<
+  ValueType = any,
+  DataType extends DefaultItem = any
+> extends StateObserversInterface<ValueType> {
   /**
    * Observer responsible for the output of the Group.
    */

packages/core/src/collection/item/index.ts

@@ -5,8 +5,8 @@ import {
   StateKey,
   StateRuntimeJobConfigInterface,
 } from '../../state';
-import { Collection, DefaultItem } from '../collection';
-import { SelectorKey } from '../selector';
+import type { Collection, DefaultItem } from '../collection';
+import type { SelectorKey } from '../selector';
 import { CollectionPersistent } from '../collection.persistent';
 
 export class Item<

packages/core/src/collection/public/createCollection.ts

@@ -1,5 +1,5 @@
 import { Collection, CreateCollectionConfig, DefaultItem } from '../collection';
-import { Agile } from '../../agile';
+import type { Agile } from '../../agile';
 import { shared } from '../../shared';
 
 /**