Skip to content

Conversation

ahmadawais
Copy link
Member

Adds config file based env vars to support Cloudflare and other envs where process.env is not available.

import pipeSummary from '../baseai/pipes/summary';
import {Pipe} from './../../../packages/core/src/pipes/pipes';
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be imported from core. This will break.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Replace it with
import {Pipe} from '@baseai/core';

@@ -29,4 +29,16 @@ export interface BaseAIConfig {
log: LoggerConfig;
memory: MemoryConfig;
envFilePath: string;
env: {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be optional

// Other default configuration options can be added here
env: {} as any
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should not add it to default config as it is optional.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants