fix(types): restrict regular styles function call to one argument

[100terres]

What's included

fix: #37
fix: #15

Tag functions are properly typed and regular function calls are restrict to one argument (see: #38 (comment))

• keyframes
• glob
• css
• createGlobalStyles
• The return of the styled(...)

We can now override the types to mark the theme as always available on the props. (see: #38 (comment))

[100terres]

This is would allow us to override the props in a project with a theme with something like this:

// in a styled.d.ts file
declare module "solid-styled-components" {
  export interface DefaultTheme {
    // ...
  }

  // The theme will always be present
  export interface ThemeProp {
    theme: DefaultTheme;
    //   ^
    // The `?` was removed 
  }

We won't need to make sure that the theme exist on the props e.g.

const Button = styled("button")((props) => ({
  color: props.theme.typography.color,
  //          ^
  // we won't need to check if the theme is present or not
);

[100terres]

I could extract this in an other PR if needed.

[100terres]

This is not possible with the current version of TypeScript. I've added a comment on an existing issue on the TypeScript repo: microsoft/TypeScript#33699 (comment)

[100terres]

For now using the non-null assertion operator should do the trick:

styled("span")`
  color: ${(props) => props.theme!.palette}
`;

[100terres]

By having multiple declarations we can restrict the use of multiple arguments to the "tag function" and make sure that regular function call are restircted to one argument.

// allowed
css`
  color: ${theme.typography.color};
`
// allowed
css({
  color: ${theme.typography.color},
});

// With the current implementation the following is allow
// but with this PR the following would be disallowed
css(
  {
    color: theme.typography.color,
  },
  // When we are not using a tag function,
  // only the first argument is used by goober
  // The following won't be added to goober
  {
    font-family: theme.typography.fontFamily,
  },
);

[ryansolid]

Thank you

[100terres]

I think the theme is available in the createGlobalStyles. I'll have a look and see if it can be added.