Using Models with Commands :: getting IoC Error

[AddoSolutions]

Hello, I am looking to use some code that I have written in another part of my application that uses one or more models.

Example:

import { BaseCommand } from '@adonisjs/ace'
import { Kernel } from '@adonisjs/ace'
import { inject } from '@adonisjs/fold'
//import { DatabaseContract } from '@ioc:Adonis/Lucid/Database'
import { ApplicationContract } from '@ioc:Adonis/Core/Application'
import Crawl from "../app/Helpers/Crawl";


@inject([null, null, 'Adonis/Lucid/Database'])
@inject([null, null, 'Adonis/Lucid/Orm'])
export default class CrawlJob extends BaseCommand {

  public static needsApplication = true
  constructor (app: ApplicationContract, kernel: Kernel, private db: DatabaseContract) {
    super(app, kernel)
  }

  public static settings = {
    loadApp: true,
  }


  /**
	 * Command Name is used to run the command
	 */
  public static commandName = 'crawl:job'

	/**
	 * Command Name is displayed in the "help" output
	 */
  public static description = ''

  public async run () {

    let crawl = new Crawl({taskId: 1});

    try {
      await crawl.start();
    }catch(e){
      console.error(e);
    }

    console.log(crawl)

  }
}

Inside of Crawl, some code is run, and includes a Model, to which I get the following error:

IocLookupException: E_IOC_LOOKUP_FAILED: Cannot resolve "Adonis/Lucid/Orm" namespace from the IoC Container

at app/Models/Task.ts(anonymous):2
1  import { DateTime } from 'luxon'
2  import {BaseModel, BelongsTo, belongsTo, column, ManyToMany, manyToMany} from '@ioc:Adonis/Lucid/Orm'
3  import User from "App/Models/User";
4  import Person from "App/Models/Person";

Any ideas how to get past this?

[thetutlage]

Not sure what is inside the app/Helpers/Crawl file. You cannot have top level imports inside your command that relies on the application code, because when the commands are scanned, the application is not ready. So I suggest moving everything inside the handle method.

public async run () {
   const Crawl = (await import('../app/Helpers/Crawl')).default  
}

Also, there is no need to inject values to the constructor unless you are deciding to provide different values when writing the tests. Simply using imports directly.

public async run () {
   const Db = (await import('Adonis/Lucid/Database')).default
   const Orm = (await import('Adonis/Lucid/Orm')).default
}

[AddoSolutions]

The problem is that there is a series of shared classes inside of Crawl (actually a whole series of code) that references Models and other application code.

Is there no way to "fully load" the application before continuing so I can run the remaining code "normally"?

As an example, I figured out a very hacky workaround, but illustrates what I am looking to do, I just want to load the full application (ideally except for the HTTP server itself)

import {BaseCommand} from '@adonisjs/ace'
import {Kernel} from '@adonisjs/ace'
import {DatabaseContract} from '@ioc:Adonis/Lucid/Database'
import {ApplicationContract} from '@ioc:Adonis/Core/Application'
import getPort from 'get-port'

export default class CrawlJob extends BaseCommand {

  public static needsApplication = true

  constructor(app: ApplicationContract, kernel: Kernel, private db: DatabaseContract) {
    super(app, kernel)
  }

  public static settings = {
    loadApp: true,
  }


  /**
   * Command Name is used to run the command
   */
  public static commandName = 'crawl:job'

  /**
   * Command Name is displayed in the "help" output
   */
  public static description = ''

  /**
   * This is stupidly hacky, not 100% sure how I would leverage Adonis to clean this up, but
   * is more to workaround the issue that I am having with getting the application to load
   */
  public async startApplication() {
    const {Ignitor} = await import('@adonisjs/core/build/src/Ignitor')
    process.env.PORT = String(await getPort())
    await new Ignitor(process.env.ADONIS_ACE_CWD).httpServer().start()
  }

  public async run() {

    await this.startApplication();

    let Crawl = (await import("../app/Helpers/Crawl")).default;
    let crawl = new Crawl({taskId: 1});

    try {
      await crawl.start();
    } catch (e) {
      console.error(e);
    }

    console.log(crawl)

  }
}

[AddoSolutions]

To make things more confusing for me, if I use the exact code from lucid/commands/DbSeed.ts (https://github.com/adonisjs/lucid/blob/develop/commands/DbSeed.ts) I get the same error using code that WORKS for the DbSeeder.

Updated startApplication():

  public async startApplication() {
    const db = this.application.container.use('Adonis/Lucid/Database')

    this.connection = this.connection || db.primaryConnectionName
    const connection = db.getRawConnection(this.connection)

    /**
     * Ensure the define connection name does exists in the
     * config file
     */
    if (!connection) {
      this.logger.error(
        `"${connection}" is not a valid connection name. Double check config/database file`
      )
      return
    }
  }

I still get:

IocLookupException: E_IOC_LOOKUP_FAILED: Cannot resolve "Adonis/Lucid/Database" namespace from the IoC Container

at ./commands/CrawlJob.ts(CrawlJob.startApplication):38
33    /**
34     * This is stupidly hacky, not 100% sure how I would leverage Adonis to clean this up, but
35     * is more to workaround the issue that I am having with getting the application to load
36     */
37    public async startApplication() {
38      const db = this.application.container.use('Adonis/Lucid/Database'). <<< This is the line highlighted
39  
40

[AddoSolutions]

Dude, node ace generate:manifest is KILLING me. The code was already correct.

import { BaseCommand, flags } from '@adonisjs/core/build/standalone'
import {Kernel} from '@adonisjs/ace'
import {DatabaseContract} from '@ioc:Adonis/Lucid/Database'
import {ApplicationContract} from '@ioc:Adonis/Core/Application'

export default class CrawlJob extends BaseCommand {

  public static needsApplication = true

  // THIS is all that is needed
  public static settings = {
    loadApp: true,
  }

  /**
   * Choose a custom pre-defined connection. Otherwise, we use the
   * default connection
   */
  @flags.string({ description: 'Define a custom database connection for the seeders', alias: 'c' })
  public connection: string


  /**
   * Command Name is used to run the command
   */
  public static commandName = 'crawl:job'

  /**
   * Command Name is displayed in the "help" output
   */
  public static description = ''


  public async run() {

    let Crawl = (await import("../app/Helpers/Crawl")).default;
    let crawl = new Crawl({taskId: 1});

    try {
      await crawl.start();
    } catch (e) {
      console.error(e);
    }

    console.log(crawl)

  }
}

In case anyone is having trouble with this going forward.

1. Add this line to your file

export default class YourCommand extends BaseCommand {

  public static needsApplication = true

  public static settings = {
    loadApp: true,
  }

2. DONT FORGET TO UPDATE YOUR MANIFEST!

👇 👇 👇 👇 👇 👇 👇 👇 👇 👇 👇 👇

👉 node ace generate:manifest 👈
👉 node ace generate:manifest 👈
👉 node ace generate:manifest 👈
👉 node ace generate:manifest 👈
👉 node ace generate:manifest 👈
👉 node ace generate:manifest 👈
👉 node ace generate:manifest 👈
👉 node ace generate:manifest 👈

☝️ ☝️ ☝️ ☝️ ☝️ ☝️ ☝️ ☝️ ☝️ ☝️ ☝️ ☝️

Hopefully this helps someone else

[robextrem]

Be proud, this helped me a lot and saved me plenty of time. ty