Skip to content

Commit

Permalink
fix(cli): manually handle shared args
Browse files Browse the repository at this point in the history
  • Loading branch information
fu050409 committed Jul 5, 2024
1 parent fe162a3 commit 688e12f
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 4 deletions.
7 changes: 5 additions & 2 deletions packages/cli/src/config.ts
Original file line number Diff line number Diff line change
@@ -1,11 +1,13 @@
import { FarmCLIOptions, UserConfig } from '@farmfe/core';
import {
FarmCLIBuildOptions,
GlobalFarmCLIOptions,
NormalizedFarmCLIBuildOptions
} from './types.js';
import { resolveCommonOptions } from './utils.js';

export function getOptionFromBuildOption(
options: NormalizedFarmCLIBuildOptions & GlobalFarmCLIOptions
options: FarmCLIBuildOptions & GlobalFarmCLIOptions
): FarmCLIOptions & UserConfig {
const {
input,
Expand All @@ -17,7 +19,8 @@ export function getOptionFromBuildOption(
sourcemap,
treeShaking,
mode
} = options;
} = resolveCommonOptions(options) as NormalizedFarmCLIBuildOptions &
GlobalFarmCLIOptions;

const output: UserConfig['compilation']['output'] = {
...(outDir && { path: outDir }),
Expand Down
8 changes: 6 additions & 2 deletions packages/cli/src/utils.ts
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,11 @@ import { Logger } from '@farmfe/core';
import spawn from 'cross-spawn';
import walkdir from 'walkdir';

import type { FarmCLICommonOptions, GlobalFarmCLIOptions } from './types.js';
import type {
FarmCLIBuildOptions,
FarmCLICommonOptions,
GlobalFarmCLIOptions
} from './types.js';

const logger = new Logger();
interface installProps {
Expand Down Expand Up @@ -186,7 +190,7 @@ export function resolveCliConfig(root: string, config: string) {
*/
export function resolveCommonOptions(
options: FarmCLICommonOptions & GlobalFarmCLIOptions
): GlobalFarmCLIOptions {
): FarmCLIBuildOptions & GlobalFarmCLIOptions {
const resolvedOptions = { ...options };
resolvedOptions.c && (resolvedOptions.config = resolvedOptions.c);
resolvedOptions.config && (resolvedOptions.c = resolvedOptions.config);
Expand Down

0 comments on commit 688e12f

Please sign in to comment.