@travetto/config 8.0.0-alpha.0 → 8.0.0-alpha.2

package/README.md

@@ -25,7 +25,7 @@ Config loading follows a defined resolution path, below is the order in increasi
    1. `resources/{role}.<ext>` - Priority `150` - Load environment specific profile configurations as defined by the values of `process.env.TRV_ROLE`. If the role is `std`, it is replaced with `local` for local development.
    1. `resources/*.<ext>` - Priority `200` - Load profile specific configurations as defined by the values in `process.env.TRV_PROFILES`
    1. [@Injectable](https://github.com/travetto/travetto/tree/main/module/di/src/decorator.ts#L16) [ConfigSource](https://github.com/travetto/travetto/tree/main/module/config/src/source/types.ts#L11) - Priority `???` - These are custom config sources provided by the module, and are able to define their own priorities
-   1. [OverrideConfigSource](https://github.com/travetto/travetto/tree/main/module/config/src/source/override.ts#L9) - Priority `999` - This is for [@EnvVar](https://github.com/travetto/travetto/tree/main/module/config/src/decorator.ts#L37) overrides, and is at the top priority for all built-in config sources.
+   1. [OverrideConfigSource](https://github.com/travetto/travetto/tree/main/module/config/src/source/override.ts#L9) - Priority `999` - This is for [@EnvVar](https://github.com/travetto/travetto/tree/main/module/config/src/decorator.ts#L36) overrides, and is at the top priority for all built-in config sources.
 
 By default all configuration data is inert, and will only be applied when constructing an instance of a configuration class.
 
@@ -182,7 +182,7 @@ At startup, the [ConfigurationService](https://github.com/travetto/travetto/tree
 The [ConfigurationService](https://github.com/travetto/travetto/tree/main/module/config/src/service.ts#L24) service provides injectable access to all of the loaded configuration. For simplicity, a decorator, [@Config](https://github.com/travetto/travetto/tree/main/module/config/src/decorator.ts#L13) allows for classes to automatically be bound with config information on post construction via the [Dependency Injection](https://github.com/travetto/travetto/tree/main/module/di#readme "Dependency registration/management and injection support.") module. The decorator will install a `postConstruct` method if not already defined, that performs the binding of configuration.  This is due to the fact that we cannot rewrite the constructor, and order of operation matters.
 
 ### Environment Variables
-Additionally there are times in which you may want to also support configuration via environment variables.  [@EnvVar](https://github.com/travetto/travetto/tree/main/module/config/src/decorator.ts#L37) supports override configuration values when environment variables are present. 
+Additionally there are times in which you may want to also support configuration via environment variables.  [@EnvVar](https://github.com/travetto/travetto/tree/main/module/config/src/decorator.ts#L36) supports override configuration values when environment variables are present. 
 
 The decorator takes in a namespace, of what part of the resolved configuration you want to bind to your class. Given the following class:
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@travetto/config",
-  "version": "8.0.0-alpha.0",
+  "version": "8.0.0-alpha.2",
   "type": "module",
   "description": "Configuration support",
   "keywords": [
@@ -27,8 +27,8 @@
     "directory": "module/config"
   },
   "dependencies": {
-    "@travetto/di": "^8.0.0-alpha.0",
-    "@travetto/schema": "^8.0.0-alpha.0",
+    "@travetto/di": "^8.0.0-alpha.2",
+    "@travetto/schema": "^8.0.0-alpha.2",
     "yaml": "^2.8.2"
   },
   "travetto": {

package/src/decorator.ts

@@ -17,15 +17,14 @@ export function Config(namespace: string) {
 
     ConfigOverrideUtil.setOverrideConfig(cls, namespace);
 
-    DependencyRegistryIndex.getForRegister(cls).registerClass();
-
-    const handle: Function = cls.prototype.postConstruct;
-    cls.prototype.postConstruct = async function (): Promise<void> {
-      // Apply config
-      const config = await DependencyRegistryIndex.getInstance(ConfigurationService);
-      await config.bindTo(cls, this, namespace);
-      await handle?.call(this);
-    };
+    DependencyRegistryIndex.registerClass(cls);
+    DependencyRegistryIndex.registerPostConstruct(cls, {
+      priority: 0,
+      async operation(this: ClassInstance): Promise<void> {
+        const config = await DependencyRegistryIndex.getInstance(ConfigurationService);
+        await config.bindTo(cls, this, namespace);
+      }
+    });
     return cls;
   };
 }

package/src/parser/parser.ts

@@ -1,7 +1,7 @@
 import fs from 'node:fs/promises';
 import path from 'node:path';
 
-import { DependencyRegistryIndex, Injectable } from '@travetto/di';
+import { DependencyRegistryIndex, Injectable, PostConstruct } from '@travetto/di';
 import { RuntimeError, toConcrete } from '@travetto/runtime';
 
 import type { ConfigData, ConfigParser } from './types.ts';
@@ -12,7 +12,8 @@ export class ParserManager {
   #extMatch: RegExp;
   #parsers: Record<string, ConfigParser>;
 
-  async postConstruct(): Promise<void> {
+  @PostConstruct()
+  async initializeParsers(): Promise<void> {
     const parsers = await DependencyRegistryIndex.getInstances(toConcrete<ConfigParser>());
 
     // Register parsers

package/src/service.ts

@@ -1,7 +1,7 @@
 import util from 'node:util';
 
 import { RuntimeError, toConcrete, castTo, type Class, Env, Runtime, RuntimeResources, getClass } from '@travetto/runtime';
-import { DependencyRegistryIndex, getDefaultQualifier, Injectable } from '@travetto/di';
+import { DependencyRegistryIndex, getDefaultQualifier, Injectable, PostConstruct } from '@travetto/di';
 import { BindUtil, DataUtil, SchemaRegistryIndex, SchemaValidator, ValidationResultError } from '@travetto/schema';
 
 import { ParserManager } from './parser/parser.ts';
@@ -49,7 +49,8 @@ export class ConfigurationService {
    *  - When dealing with two profiles of the same name, they are then sorted by priority
    *  - If of the same priority, then alpha sort on the source
    */
-  async postConstruct(): Promise<void> {
+  @PostConstruct()
+  async loadConfigurations(): Promise<void> {
     const providers = DependencyRegistryIndex.getCandidates(toConcrete<ConfigSource>());
 
     const configs = await Promise.all(