Skip to content

Recent commit breaks the build #48

Open
@okigan

Description

@okigan

@sts07142 the 2138bbcdc3071e9e2413a125d107332723a3890a change breaks the build:

$ npm install && npm run build                         

up to date, audited 419 packages in 632ms

133 packages are looking for funding
  run `npm fund` for details

9 vulnerabilities (1 low, 5 moderate, 3 high)

To address issues that do not require attention, run:
  npm audit fix

To address all issues (including breaking changes), run:
  npm audit fix --force

Run `npm audit` for details.

> @context-labs/autodoc@0.0.9 build
> tsc

src/cli/commands/index/processRepository.ts:124:41 - error TS2345: Argument of type 'LLMModels' is not assignable to parameter of type 'TiktokenModel'.

124     const encoding = encoding_for_model(model.name);
                                            ~~~~~~~~~~

src/cli/commands/index/selectModel.ts:78:41 - error TS2345: Argument of type 'LLMModels' is not assignable to parameter of type 'TiktokenModel'.

78     const encoding = encoding_for_model(model);
                                           ~~~~~


Found 2 errors in 2 files.

Errors  Files
     1  src/cli/commands/index/processRepository.ts:124
     1  src/cli/commands/index/selectModel.ts:78

Activity

linked a pull request that will close this issue on Aug 10, 2024
sts07142

sts07142 commented on Aug 10, 2024

@sts07142
Contributor

#49
We need to upgrade version @dqbd/tiktoken.
@dqbd/tiktoken 1.0.2 -> 1.0.15

salimmj

salimmj commented on Aug 11, 2024

@salimmj

#49 We need to upgrade version @dqbd/tiktoken. @dqbd/tiktoken 1.0.2 -> 1.0.15

I don't think this fixes it

sts07142

sts07142 commented on Aug 12, 2024

@sts07142
Contributor

#49
fix: Add function to convert model's name when using gpt-4o-mini.

@dqbd/tiktoken encoding_for_model( )
^ It doesn't have gpt-4o-mini's encoding model, but same encoding model as gpt-4o (o200k_base)

Fix errors in Errors Files
1 src/cli/commands/index/processRepository.ts:124
1 src/cli/commands/index/selectModel.ts:78

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

      Development

      Participants

      @okigan@salimmj@sts07142

      Issue actions

        Recent commit breaks the build · Issue #48 · context-labs/autodoc