feat(context): allows default namespace for bindings from classes

docs/site/Binding.md

@@ -461,6 +461,8 @@ parameter of `BindingFromClassOptions` type with the following settings:
   }
   ```
 
+- defaultNamespace: Default namespace if namespace or namespace tag does not
+  exist
 - defaultScope: Default scope if the binding does not have an explicit scope
   set. The `scope` from `@bind` of the bound class takes precedence.
 

packages/context/src/__tests__/unit/binding-inspector.unit.ts

@@ -14,6 +14,7 @@ import {
   createBindingFromClass,
   Provider,
 } from '../..';
+import {ContextTags} from '../../keys';
 
 describe('createBindingFromClass()', () => {
   it('inspects classes', () => {
@@ -206,6 +207,33 @@ describe('createBindingFromClass()', () => {
     expect(binding.key).to.eql('services.MyService');
   });
 
+  it('honors default namespace with options', () => {
+    class MyService {}
+
+    @bind({tags: {[ContextTags.NAMESPACE]: 'my-services'}})
+    class MyServiceWithNS {}
+
+    const ctx = new Context();
+    let binding = givenBindingFromClass(MyService, ctx, {
+      defaultNamespace: 'services',
+    });
+
+    expect(binding.key).to.eql('services.MyService');
+
+    binding = givenBindingFromClass(MyService, ctx, {
+      namespace: 'my-services',
+      defaultNamespace: 'services',
+    });
+
+    expect(binding.key).to.eql('my-services.MyService');
+
+    binding = givenBindingFromClass(MyServiceWithNS, ctx, {
+      defaultNamespace: 'services',
+    });
+
+    expect(binding.key).to.eql('my-services.MyServiceWithNS');
+  });
+
   it('includes class name in error messages', () => {
     expect(() => {
       // Reproduce a problem that @bajtos encountered when the project

packages/context/src/binding-inspector.ts

@@ -201,17 +201,23 @@ export type BindingFromClassOptions = {
    */
   type?: string;
   /**
-   * Artifact name, such as `my-rest-server` and `my-controller`
+   * Artifact name, such as `my-rest-server` and `my-controller`. It
+   * overrides the name tag
    */
   name?: string;
   /**
-   * Namespace for the binding key, such as `servers` and `controllers`
+   * Namespace for the binding key, such as `servers` and `controllers`. It
+   * overrides the default namespace or namespace tag
    */
   namespace?: string;
   /**
    * Mapping artifact type to binding key namespaces
    */
   typeNamespaceMapping?: TypeNamespaceMapping;
+  /**
+   * Default namespace if the binding does not have an explicit namespace
+   */
+  defaultNamespace?: string;
   /**
    * Default scope if the binding does not have an explicit scope
    */
@@ -313,7 +319,9 @@ function buildBindingKey<T>(
   if (key) return key;
 
   let namespace =
-    options.namespace ?? bindingTemplate.tagMap[ContextTags.NAMESPACE];
+    options.namespace ??
+    bindingTemplate.tagMap[ContextTags.NAMESPACE] ??
+    options.defaultNamespace;
   if (!namespace) {
     const namespaces = Object.assign(
       {},