er

Author: kaspernielsen

TODO.txt

@@ -1,7 +1,10 @@
 Major Architectual
   Hooks
   Namespaces/Extensions
-  Lifecycle
+  Lifetime
+
+Cleanup
+  Services (incl .provide)
 
 
 

modules/packed/src/main/java/app/packed/Framework.java

@@ -54,6 +54,7 @@ public static Framework.Version version() {
     // Typically called from a static init of the class
 
     // Tror vi skal have dem mere specifikke fx  Bean.openClass
+    // Or maybe BeanOpener
     @Deprecated
     public static void openClassToFramework(MethodHandles.Lookup lookup) {
         openClassToFramework(lookup, lookup.lookupClass());

modules/packed/src/main/java/app/packed/bean/BeanConfiguration.java

@@ -9,7 +9,6 @@
 import java.util.stream.Stream;
 
 import app.packed.binding.Key;
-import app.packed.build.action.BuildActionable;
 import app.packed.component.ComponentConfiguration;
 import app.packed.component.ComponentRealm;
 import app.packed.context.Context;
@@ -47,7 +46,6 @@ public BeanConfiguration(BeanHandle<?> handle) {
      * @throws UnsupportedOperationException
      *             if called on a bean with void bean class
      */
-    @BuildActionable("bean.allowMultiClass")
     public BeanConfiguration<T> allowMultiClass() {
         checkIsConfigurable();
         handle.allowMultiClass();
@@ -102,7 +100,6 @@ public final BeanKind beanLifetime() {
      * @see CodeGenerated
      * @see BindableVariable#bindGeneratedConstant(Supplier)
      */
-    @BuildActionable("bean.addCodeGenerator")
     public <K> void bindComputedConstant(Class<K> key, Supplier<? extends K> supplier) {
         checkIsConfigurable();
         bindComputedConstant(Key.of(key), supplier);
@@ -134,7 +131,6 @@ public <K> void bindComputedConstant(Class<K> key, Supplier<? extends K> supplie
      * @see ComputedConstant
      * @see BindableVariable#bindGeneratedConstant(Supplier)
      */
-    @BuildActionable("bean.addCodeGenerator")
     // ComputedService instead??? buildConstant??? IDK
     // Hmm, bliver vi noedt til at sige noget om hvornaar den bliver kaldt???
     // Den bliver jo tidligst kaldt som en del af build processen
@@ -235,7 +231,6 @@ public final boolean isInSameContainer(BeanConfiguration<?> other) {
      *             if there is another bean with the same name in the container. Or if the container has a child container
      *             with the same name.
      */
-    @BuildActionable("bean.named")
     public BeanConfiguration<?> named(String name) {
         checkIsConfigurable();
         handle.named(name);
@@ -310,7 +305,6 @@ protected Set<Key<?>> requestedServiceKeys() {
 
     /** {@inheritDoc} */
     @Override
-    @BuildActionable("component.addTags") // Hmm or bean.addTags
     public BeanConfiguration<T> tag(String... tags) {
         checkIsConfigurable();
         handle.componentTag(tags);

modules/packed/src/main/java/app/packed/bean/BeanMirror.java

@@ -16,7 +16,6 @@
 import app.packed.bean.BeanTrigger.AutoServiceInheritable;
 import app.packed.binding.Key;
 import app.packed.build.Mirror;
-import app.packed.build.action.BuildActionMirror;
 import app.packed.component.ComponentMirror;
 import app.packed.component.ComponentPath;
 import app.packed.component.ComponentRealm;
@@ -149,10 +148,6 @@ public final int hashCode() {
         return handle.bean.hashCode();
     }
 
-    BuildActionMirror installationAction() {
-        throw new UnsupportedOperationException();
-    }
-
     /** {@return the extension that installed the bean, typically BaseExtension} */
     public final Class<? extends Extension<?>> installedByExtension() {
         return handle.bean.installedBy.extensionType;

…packed/bean/sandbox/BeanTriggerType.java …packed/bean/sandbox/BeanTriggerKind.javamodules/packed/src/main/java/app/packed/bean/sandbox/BeanTriggerType.java renamed to modules/packed/src/main/java/app/packed/bean/sandbox/BeanTriggerKind.java modules/packed/src/main/java/app/packed/bean/sandbox/BeanTriggerType.java renamed to modules/packed/src/main/java/app/packed/bean/sandbox/BeanTriggerKind.java

@@ -24,6 +24,6 @@
 // Hmm, maaske bare annoteringstypen????
 
 // Maybe skip Inheritable... It is only source
-enum BeanTriggerType {
-    ANNOTATED_CLASS, ANNOTATED_METHOD, ANNOTATED_FIELD, ANNOTATED_VARIABLE, CONTEXTUAL_SERVICE, INHERITABLE_CONTEXTUAL_SERVICE
+enum BeanTriggerKind {
+    ANNOTATED_CLASS, ANNOTATED_METHOD, ANNOTATED_FIELD, ANNOTATED_VARIABLE, AUTO_SERVICE, INHERITABLE_CONTEXTUAL_SERVICE
 }

modules/packed/src/main/java/app/packed/bean/sandbox/NewIntrospector.java

@@ -62,7 +62,7 @@ public final ComponentPath componentPath() {
     /** {@inheritDoc} */
     @Override
     public final boolean equals(Object obj) {
-        return obj instanceof ComponentConfiguration cc && handle().equals(cc.handle());
+        return obj instanceof ComponentConfiguration cc && handle() == cc.handle();
     }
 
     /** {@return the underlying handle of this component} */

modules/packed/src/main/java/app/packed/component/ComponentConfiguration.java

@@ -7,7 +7,6 @@
 import java.util.stream.Stream;
 
 import app.packed.bean.BeanConfiguration;
-import app.packed.build.action.BuildActionable;
 import app.packed.component.ComponentConfiguration;
 import app.packed.extension.Extension;
 import internal.app.packed.extension.ExtensionSetup;
@@ -46,7 +45,6 @@ public final <T extends BeanConfiguration<?>> Stream<T> beans(Class<? extends Be
 
     /** {@inheritDoc} */
     @Override
-    @BuildActionable("container.addTags")
     public ComponentConfiguration tag(String... tags) {
         checkIsConfigurable();
         handle.componentTag(tags);
@@ -113,7 +111,6 @@ public final boolean isExtensionUsed(Class<? extends Extension<?>> extensionType
      *             characters and '_', '-' or '.'
      * @see Wirelet#named(String)
      */
-    @BuildActionable("container.named")
     public ContainerConfiguration named(String name) {
         checkIsConfigurable();
         handle.container.named(name);
@@ -177,7 +174,6 @@ public String toString() {
      * @see #extensionsTypes()
      * @see BaseAssembly#use(Class)
      */
-    @BuildActionable("container.installExtension")
     public final <E extends Extension<?>> E use(Class<E> extensionClass) {
         ExtensionSetup extension = handle.container.useExtension(extensionClass, null);
         return extensionClass.cast(extension.instance());

modules/packed/src/main/java/app/packed/container/ContainerConfiguration.java

@@ -16,7 +16,6 @@
 package app.packed.container;
 
 import app.packed.assembly.Assembly;
-import app.packed.binding.Key;
 import internal.app.packed.container.PackedContainerInstaller;
 
 /**
@@ -113,25 +112,6 @@ public sealed interface ContainerInstaller<H extends ContainerHandle<?>> permits
      */
     ContainerInstaller<H> named(String name);
 
-    /**
-     * Provides constants per Carrier Instance for this particular container builder
-     *
-     * @param <T>
-     * @param key
-     * @param arg
-     * @return
-     *
-     * @see ExtensionLink#ofConstant(Class, Object)
-     */
-    default <T> ContainerInstaller<H> provideGuestConstant(Class<T> key, T constant) {
-        return provideGuestConstant(Key.of(key), constant);
-    }
-
-    /**
-     * @see FromLifetimeChannel
-     */
-    <T> ContainerInstaller<H> provideGuestConstant(Key<T> key, T constant);
-
     /**
      * Sets the value of the specified container local for the container being built.
      *