fbredius/storybook

Function applyPresets has 38 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function applyPresets(
  presets: LoadedPreset[],
  extension: string,
  config: any,
  args: any,

Function selectStory has 38 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    selectStory: (kindOrId = undefined, story = undefined, options = {}) => {
      const { ref, viewMode: viewModeFromArgs } = options;
      const {
        viewMode: viewModeFromState = 'story',
        storyId,

Function eventToShortcut has 38 lines of code (exceeds 25 allowed). Consider refactoring.

Open

export const eventToShortcut = (e: KeyboardEvent): KeyCollection | null => {
  // Meta key only doesn't map to a shortcut
  if (['Meta', 'Alt', 'Control', 'Shift'].includes(e.key)) {
    return null;
  }

Function normalizeStory has 38 lines of code (exceeds 25 allowed). Consider refactoring.

Open

export function normalizeStory<TFramework extends AnyFramework>(
  key: StoryId,
  storyAnnotations: LegacyStoryAnnotationsOrFn<TFramework>,
  meta: ComponentAnnotations<TFramework>
): NormalizedStoryAnnotations<TFramework> {

Function executeLoadable has 38 lines of code (exceeds 25 allowed). Consider refactoring.

Open

export function executeLoadable(loadable: Loadable) {
  let reqs = null;
  // todo discuss / improve type check
  if (Array.isArray(loadable)) {
    reqs = loadable;

Identical blocks of code found in 2 locations. Consider refactoring.

Open

  const defaultExportWithTitle = root
    .find(j.ExportDefaultDeclaration)
    .filter((def) => def.node.declaration.properties.map((p) => p.key.name).includes('title'));

• Read up
  Read up

Similar blocks of code found in 4 locations. Consider refactoring.

Open

ForwardRefButtonRender.propTypes = {
  /** Boolean indicating whether the button should render as disabled */
  disabled: PropTypes.bool,
  /** button label. */
  label: PropTypes.string.isRequired,

• Read up
  Read up

Similar blocks of code found in 4 locations. Consider refactoring.

Open

ForwardRefButtonOuterPropTypes.propTypes = {
  /** Boolean indicating whether the button should render as disabled */
  disabled: PropTypes.bool,
  /** button label. */
  label: PropTypes.string.isRequired,

• Read up
  Read up

Identical blocks of code found in 2 locations. Consider refactoring.

Open

  root
    .find(j.ExportDefaultDeclaration)
    .filter((def) => def.node.declaration.properties.map((p) => p.key.name).includes('title'))

• Read up
  Read up

Similar blocks of code found in 4 locations. Consider refactoring.

Open

NamedExportButton.propTypes = {
  /** Boolean indicating whether the button should render as disabled */
  disabled: PropTypes.bool,
  /** button label. */
  label: PropTypes.string.isRequired,

• Read up
  Read up

Similar blocks of code found in 4 locations. Consider refactoring.

Open

BaseButton.propTypes = {
  /** Boolean indicating whether the button should render as disabled */
  disabled: PropTypes.bool,
  /** button label. */
  label: PropTypes.string.isRequired,

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

Header.propTypes = {
  user: PropTypes.shape({}),
  onLogin: PropTypes.func.isRequired,
  onLogout: PropTypes.func.isRequired,
  onCreateAccount: PropTypes.func.isRequired,

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

Page.propTypes = {
  user: PropTypes.shape({}),
  onLogin: PropTypes.func.isRequired,
  onLogout: PropTypes.func.isRequired,
  onCreateAccount: PropTypes.func.isRequired,

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

Header.propTypes = {
  user: PropTypes.shape({}),
  onLogin: PropTypes.func.isRequired,
  onLogout: PropTypes.func.isRequired,
  onCreateAccount: PropTypes.func.isRequired,

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

  it('adds "./" to a normalized relative path', () => {
    const filename = path.join('src', 'Comp.story.js');
    expect(normalizeStoryPath(filename)).toEqual(`.${path.sep}${filename}`);
  });

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

      alias: {
        ...(features?.emotionAlias ? themingPaths : {}),
        ...storybookPaths,
        react: path.dirname(require.resolve('react/package.json')),
        'react-dom': path.dirname(require.resolve('react-dom/package.json')),

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

      alias: {
        ...(features?.emotionAlias ? themingPaths : {}),
        ...storybookPaths,
        react: path.dirname(require.resolve('react/package.json')),
        'react-dom': path.dirname(require.resolve('react-dom/package.json')),

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

function interopRequireDefault(filePath: string) {
  // eslint-disable-next-line global-require,import/no-dynamic-require
  const result = require(filePath);

  const isES6DefaultExported =

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

function interopRequireDefault(filePath: string) {
  // eslint-disable-next-line import/no-dynamic-require,global-require
  const result = require(filePath);

  const isES6DefaultExported =

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

          {groups.ungrouped.map((row) => (
            <ArgRow key={row.key} row={row} arg={args && args[row.key]} {...common} />
          ))}

• Read up
  Read up