Update per review feedback

README.md

@@ -297,9 +297,6 @@ in the root container, and would always have the `currentUser` from the first
 request. Modules should generally not have a longer lifetime than their
 dependencies, as this can cause issues of stale data.
 
-If you want a mismatched configuration like the above to error, set
-`errorOnShorterLivedDependencies` in the container options.
-
 ```js
 const makePrintTime = ({ time }) => () => {
   console.log('Time:', time)
@@ -324,6 +321,11 @@ container.resolve('printTime')()
 container.resolve('printTime')()
 ```
 
+If you want a mismatched configuration like this to error, set
+`errorOnShorterLivedDependencies` in the container options. This will trigger
+the following error at runtime when the singleton `printTime` is resolved:
+`AwilixResolutionError: Could not resolve 'time'. Dependency 'time' has a shorter lifetime than its ancestor: 'printTime'`
+
 Read the documentation for [`container.createScope()`](#containercreatescope)
 for more examples.
 

src/__tests__/container.test.ts

@@ -466,7 +466,7 @@ describe('container', () => {
       const err = throws(() => container.resolve('first'))
       expect(err.message).toContain('first -> second')
       expect(err.message).toContain(
-        "Dependency 'second' has a shorter lifetime than its parent: 'first'",
+        "Dependency 'second' has a shorter lifetime than its ancestor: 'first'",
       )
     })
 
@@ -516,7 +516,7 @@ describe('container', () => {
       const err = throws(() => container.resolve('first'))
       expect(err.message).toContain('first -> second')
       expect(err.message).toContain(
-        "Dependency 'second' has a shorter lifetime than its parent: 'first'",
+        "Dependency 'second' has a shorter lifetime than its ancestor: 'first'",
       )
     })
 

src/container.ts

@@ -512,7 +512,7 @@ export function createContainer<T extends object = any, U extends object = any>(
           throw new AwilixResolutionError(
             name,
             resolutionStack,
-            `Dependency '${name.toString()}' has a shorter lifetime than its parent: '${resolutionStack[
+            `Dependency '${name.toString()}' has a shorter lifetime than its ancestor: '${resolutionStack[
               maybeLongerLifetimeParentIndex
             ].name.toString()}'`,
           )

src/lifetime.ts

@@ -26,6 +26,9 @@ export const Lifetime: Record<LifetimeType, LifetimeType> = {
   SCOPED: 'SCOPED',
 }
 
+/**
+ * Returns true if and only if the first lifetime is strictly longer than the second.
+ */
 export function isLifetimeLonger(a: LifetimeType, b: LifetimeType): boolean {
   return (
     (a === Lifetime.SINGLETON && b !== Lifetime.SINGLETON) ||

src/resolvers.ts

@@ -133,7 +133,8 @@ export type Constructor<T> = { new (...args: any[]): T }
 export function asValue<T>(value: T, opts?: ResolverOptions<T>): Resolver<T> {
   return {
     resolve: () => value,
-    ...{ lifetime: Lifetime.SCOPED, ...opts },
+    lifetime: Lifetime.SCOPED,
+    ...opts,
   }
 }