@alloy-js/csharp 0.20.0-dev.1 → 0.20.0-dev.10

package/src/components/interface/method.test.tsx

@@ -3,7 +3,6 @@ import { Children } from "@alloy-js/core/jsx-runtime";
 import { describe, expect, it } from "vitest";
 import { TestNamespace } from "../../../test/utils.jsx";
 import { Attribute } from "../attributes/attributes.jsx";
-import { SourceFile } from "../SourceFile.jsx";
 import { TypeParameterProps } from "../type-parameters/type-parameter.jsx";
 import { InterfaceDeclaration } from "./declaration.jsx";
 import { InterfaceMethod } from "./method.jsx";
@@ -28,7 +27,7 @@ describe("modifiers", () => {
         ).toRenderTo(`
         public interface TestInterface
         {
-          ${accessModifier} void MethodOne();
+            ${accessModifier} void MethodOne();
         }
       `);
       },
@@ -44,7 +43,7 @@ describe("modifiers", () => {
       ).toRenderTo(`
         public interface TestInterface
         {
-          ${methodModifier} void MethodOne();
+            ${methodModifier} void MethodOne();
         }
       `);
     });
@@ -58,7 +57,7 @@ describe("modifiers", () => {
     ).toRenderTo(`
         public interface TestInterface
         {
-          public new Task MethodOne();
+            public new Task MethodOne();
         }
       `);
   });
@@ -72,7 +71,7 @@ it("applies PascalCase naming policy", () => {
   ).toRenderTo(`
     public interface TestInterface
     {
-      void MethodOne();
+        void MethodOne();
     }
 `);
 });
@@ -102,7 +101,7 @@ it("defines params and return type", () => {
   expect(res).toRenderTo(`
     public interface TestInterface
     {
-      public string MethodOne(int intParam, string stringParam);
+        public string MethodOne(int intParam, string stringParam);
     }
   `);
 });
@@ -128,7 +127,7 @@ it("defines optional param", () => {
   expect(res).toRenderTo(`
     public interface TestInterface
     {
-      public string MethodOne(int? intParam);
+        public string MethodOne(int? intParam);
     }
   `);
 });
@@ -154,7 +153,7 @@ it("defines optional param with default", () => {
   expect(res).toRenderTo(`
     public interface TestInterface
     {
-      public string MethodOne(int intParam = 12);
+        public string MethodOne(int intParam = 12);
     }
   `);
 });
@@ -169,8 +168,8 @@ it("specify doc comment", () => {
   ).toRenderTo(`
     interface Test
     {
-      /// This is a test
-      void Method();
+        /// This is a test
+        void Method();
     }
   `);
 });
@@ -183,8 +182,8 @@ it("specify attributes", () => {
   ).toRenderTo(`
     public interface TestInterface
     {
-      [Test]
-      void Test();
+        [Test]
+        void Test();
     }
   `);
 });
@@ -204,30 +203,25 @@ describe("with type parameters", () => {
 
     expect(
       <TestNamespace>
-        <SourceFile path="TestFile.cs">
-          <InterfaceDeclaration public name="TestInterface">
-            <InterfaceMethod
-              name="Test"
-              public
-              typeParameters={typeParameters}
-              parameters={[
-                {
-                  name: "paramA",
-                  type: typeParameters[0].refkey,
-                },
-              ]}
-              returns={typeParameters[0].refkey}
-            />
-          </InterfaceDeclaration>
-        </SourceFile>
+        <InterfaceDeclaration public name="TestInterface">
+          <InterfaceMethod
+            name="Test"
+            public
+            typeParameters={typeParameters}
+            parameters={[
+              {
+                name: "paramA",
+                type: typeParameters[0].refkey,
+              },
+            ]}
+            returns={typeParameters[0].refkey}
+          />
+        </InterfaceDeclaration>
       </TestNamespace>,
     ).toRenderTo(`
-      namespace TestCode
+      public interface TestInterface
       {
-          public interface TestInterface
-          {
-              public T Test<T, U>(T paramA);
-          }
+          public T Test<T, U>(T paramA);
       }
     `);
   });
@@ -253,12 +247,12 @@ describe("with type parameters", () => {
     ).toRenderTo(`
       public interface TestInterface
       {
-        public void Test<T, U>()
-          where T : IFoo
-          where U : IBar
-        {
-          // Body
-        }
+          public void Test<T, U>()
+              where T : IFoo
+              where U : IBar
+          {
+              // Body
+          }
       }
     `);
   });
@@ -274,19 +268,19 @@ describe("formatting", () => {
           parameters={[
             {
               name: "message",
-              type: "Some.Quite.Long.Type.That.Will.Split",
+              type: "Some.Quite.Long.Type.That.Will.Split.At.One.Hundred.Chars.Line.Width",
             },
           ]}
           typeParameters={["T"]}
-          returns="Some.Quite.Long.Type.That.Will.Split"
+          returns="Some.Quite.Long.Type.That.Will.Split.At.One.Hundred.Chars.Line.Width"
         />
       </Wrapper>,
     ).toRenderTo(`
       public interface TestInterface
       {
-        public Some.Quite.Long.Type.That.Will.Split Handle<T>(
-          Some.Quite.Long.Type.That.Will.Split message
-        );
+          public Some.Quite.Long.Type.That.Will.Split.At.One.Hundred.Chars.Line.Width Handle<T>(
+              Some.Quite.Long.Type.That.Will.Split.At.One.Hundred.Chars.Line.Width message
+          );
       }
     `);
   });

package/src/components/interface/method.tsx

@@ -2,7 +2,7 @@ import {
   Block,
   Children,
   MemberDeclaration,
-  refkey,
+  MemberName,
   Refkey,
   Scope,
 } from "@alloy-js/core";
@@ -12,9 +12,8 @@ import {
   getAccessModifier,
   makeModifiers,
 } from "../../modifiers.js";
-import { useCSharpNamePolicy } from "../../name-policy.js";
-import { CSharpOutputSymbol } from "../../symbols/csharp-output-symbol.js";
-import { CSharpMemberScope, useCSharpScope } from "../../symbols/scopes.js";
+import { createMethodScope } from "../../scopes/factories.js";
+import { createMethodSymbol } from "../../symbols/factories.js";
 import { AttributeList, AttributesProp } from "../attributes/attributes.jsx";
 import { DocWhen } from "../doc/comment.jsx";
 import { ParameterProps, Parameters } from "../parameters/parameters.jsx";
@@ -76,23 +75,11 @@ export interface InterfaceMethodProps
 
 // a C# interface method
 export function InterfaceMethod(props: InterfaceMethodProps) {
-  const name = useCSharpNamePolicy().getName(props.name, "class-method");
-  const scope = useCSharpScope();
-  if (scope.kind !== "member" || scope.name !== "interface-decl") {
-    throw new Error(
-      "can't define an interface method outside of an interface scope",
-    );
-  }
-
-  const methodSymbol = new CSharpOutputSymbol(name, {
-    scope,
-    refkeys: props.refkey ?? refkey(props.name),
+  const methodSymbol = createMethodSymbol(props.name, {
+    refkeys: props.refkey,
   });
 
-  // scope for method declaration
-  const methodScope = new CSharpMemberScope("method-decl", {
-    owner: methodSymbol,
-  });
+  const methodScope = createMethodScope();
 
   const modifiers = computeModifiersPrefix([
     getAccessModifier(props),
@@ -105,7 +92,7 @@ export function InterfaceMethod(props: InterfaceMethodProps) {
         <DocWhen doc={props.doc} />
         <AttributeList attributes={props.attributes} endline />
         {modifiers}
-        {props.returns ?? "void"} {name}
+        {props.returns ?? "void"} <MemberName />
         {props.typeParameters && (
           <TypeParameters parameters={props.typeParameters} />
         )}

package/src/components/interface/property.test.tsx

@@ -30,7 +30,7 @@ describe("modifiers", () => {
         ).toRenderTo(`
         public interface TestInterface
         {
-          ${accessModifier} string TestProp { get; }
+            ${accessModifier} string TestProp { get; }
         }
       `);
       },
@@ -51,7 +51,7 @@ describe("modifiers", () => {
       ).toRenderTo(`
         public interface TestInterface
         {
-          ${methodModifier} string TestProp { get; }
+            ${methodModifier} string TestProp { get; }
         }
       `);
     });
@@ -65,7 +65,7 @@ describe("modifiers", () => {
     ).toRenderTo(`
         public interface TestInterface
         {
-          public new string TestProp { get; }
+            public new string TestProp { get; }
         }
       `);
   });
@@ -79,7 +79,7 @@ it("applies PascalCase naming policy", () => {
   ).toRenderTo(`
     public interface TestInterface
     {
-      string TestProp { get; }
+        string TestProp { get; }
     }
 `);
 });
@@ -92,7 +92,7 @@ it("has getter only", () => {
   ).toRenderTo(`
     public interface TestInterface
     {
-      string TestProp { get; }
+        string TestProp { get; }
     }
   `);
 });
@@ -105,7 +105,7 @@ it("has setter only", () => {
   ).toRenderTo(`
     public interface TestInterface
     {
-      string TestProp { set; }
+        string TestProp { set; }
     }
   `);
 });
@@ -117,7 +117,7 @@ it("has getter and setter", () => {
   ).toRenderTo(`
     public interface TestInterface
     {
-      string TestProp { get; set; }
+        string TestProp { get; set; }
     }
   `);
 });
@@ -138,8 +138,8 @@ it("specify doc comment", () => {
   ).toRenderTo(`
     interface Test
     {
-      /// This is a test
-      string Method { get; set; }
+        /// This is a test
+        string Method { get; set; }
     }
   `);
 });
@@ -158,8 +158,8 @@ it("specify attributes", () => {
   ).toRenderTo(`
     public interface TestInterface
     {
-      [Test]
-      int Test { get; set; }
+        [Test]
+        int Test { get; set; }
     }
   `);
 });

package/src/components/interface/property.tsx

@@ -3,9 +3,8 @@ import {
   Children,
   List,
   MemberDeclaration,
-  refkey,
+  MemberName,
   Refkey,
-  Scope,
 } from "@alloy-js/core";
 import {
   AccessModifiers,
@@ -13,9 +12,7 @@ import {
   getAccessModifier,
   makeModifiers,
 } from "../../modifiers.js";
-import { useCSharpNamePolicy } from "../../name-policy.js";
-import { CSharpOutputSymbol } from "../../symbols/csharp-output-symbol.js";
-import { CSharpMemberScope, useCSharpScope } from "../../symbols/scopes.js";
+import { createPropertySymbol } from "../../symbols/factories.js";
 import { AttributeList, AttributesProp } from "../attributes/attributes.jsx";
 import { DocWhen } from "../doc/comment.jsx";
 
@@ -84,22 +81,8 @@ export interface InterfacePropertyProps
  * ```
  */
 export function InterfaceProperty(props: InterfacePropertyProps) {
-  const name = useCSharpNamePolicy().getName(props.name, "class-property");
-  const scope = useCSharpScope();
-  if (scope.kind !== "member" || scope.name !== "interface-decl") {
-    throw new Error(
-      "can't define an interface method outside of an interface scope",
-    );
-  }
-
-  const propertySymbol = new CSharpOutputSymbol(name, {
-    scope,
-    refkeys: props.refkey ?? refkey(props.name),
-  });
-
-  // scope for property declaration
-  const propertyScope = new CSharpMemberScope("property-decl", {
-    owner: propertySymbol,
+  const propertySymbol = createPropertySymbol(props.name, {
+    refkeys: props.refkey,
   });
 
   const modifiers = computeModifiersPrefix([
@@ -109,19 +92,17 @@ export function InterfaceProperty(props: InterfacePropertyProps) {
   // note that scope wraps the method decl so that the params get the correct scope
   return (
     <MemberDeclaration symbol={propertySymbol}>
-      <Scope value={propertyScope}>
-        <DocWhen doc={props.doc} />
-        <AttributeList attributes={props.attributes} endline />
-        {modifiers}
-        {props.type}
-        {props.nullable && "?"} {name}{" "}
-        <Block newline inline>
-          <List joiner=" ">
-            {props.get && "get;"}
-            {props.set && "set;"}
-          </List>
-        </Block>
-      </Scope>
+      <DocWhen doc={props.doc} />
+      <AttributeList attributes={props.attributes} endline />
+      {modifiers}
+      {props.type}
+      {props.nullable && "?"} <MemberName />{" "}
+      <Block newline inline>
+        <List joiner=" ">
+          {props.get && "get;"}
+          {props.set && "set;"}
+        </List>
+      </Block>
     </MemberDeclaration>
   );
 }

package/src/components/invocation-expression/invocation-expression.test.tsx

@@ -0,0 +1,101 @@
+import { ClassDeclaration } from "#components/class/declaration.jsx";
+import { Method } from "#components/method/method.jsx";
+import { TestNamespace } from "#test/utils.jsx";
+import { namekey } from "@alloy-js/core";
+import { describe, expect, it } from "vitest";
+import { InvocationExpression } from "./invocation-expression.jsx";
+
+it("makes a call with no arguments", () => {
+  const template = <InvocationExpression target="Foo" />;
+  expect(template).toRenderTo(`Foo()`);
+});
+
+it("makes a call with arguments", () => {
+  const template = (
+    <InvocationExpression target="Foo" args={["42", `"string"`]} />
+  );
+  expect(template).toRenderTo(`Foo(42, "string")`);
+});
+
+it("makes a call with type parameters", () => {
+  const template = (
+    <InvocationExpression target="Foo" typeArgs={["Bar", "Baz"]} />
+  );
+  expect(template).toRenderTo(`Foo<Bar, Baz>()`);
+});
+
+it("makes a call to a method", () => {
+  const cls = namekey("TestClass");
+  const method = namekey("Method");
+  const template = (
+    <TestNamespace>
+      <ClassDeclaration name={cls}>
+        <Method name={method}>return 1;</Method>
+      </ClassDeclaration>
+      <hbr />
+      <InvocationExpression target={method} />;
+    </TestNamespace>
+  );
+  expect(template).toRenderTo(`
+    class TestClass
+    {
+        void Method()
+        {
+            return 1;
+        }
+    }
+    TestClass.Method();
+  `);
+});
+
+describe("formatting", () => {
+  it("doesn't break one long argument", () => {
+    const template = (
+      <InvocationExpression target="Foo" args={["oneLongArgument"]} />
+    );
+    expect(template).toRenderTo(`Foo(oneLongArgument)`, { printWidth: 10 });
+  });
+
+  it("breaks multiple arguments", () => {
+    const template = (
+      <InvocationExpression
+        target="Foo"
+        args={["oneLongArgument", "anotherLongArgument"]}
+      />
+    );
+    expect(template).toRenderTo(
+      `
+      Foo(
+        oneLongArgument,
+        anotherLongArgument
+      )
+      `,
+      { printWidth: 10 },
+    );
+  });
+
+  it("doesn't break one long type argument", () => {
+    const template = (
+      <InvocationExpression target="Foo" typeArgs={["oneLongArgument"]} />
+    );
+    expect(template).toRenderTo(`Foo<oneLongArgument>()`, { printWidth: 10 });
+  });
+
+  it("breaks multiple type arguments", () => {
+    const template = (
+      <InvocationExpression
+        target="Foo"
+        typeArgs={["oneLongArgument", "anotherLongArgument"]}
+      />
+    );
+    expect(template).toRenderTo(
+      `
+      Foo<
+        oneLongArgument,
+        anotherLongArgument
+      >()
+      `,
+      { printWidth: 10 },
+    );
+  });
+});

package/src/components/invocation-expression/invocation-expression.tsx

@@ -0,0 +1,60 @@
+import { Children, For, Indent, Show, Wrap } from "@alloy-js/core";
+
+export interface InvocationExpressionProps {
+  target: Children;
+  /**
+   * Arguments to pass to the invocation.
+   */
+  args?: Children[];
+  /**
+   * Generic type arguments for the invocation.
+   */
+  typeArgs?: Children[];
+}
+
+/**
+ * A call to a function or method.
+ *
+ * @example
+ *
+ * ```jsx
+ * <InvocationExpression target="Foo" typeArgs={["T"]} args={["x"]} />
+ * ```
+ *
+ * Renders to:
+ *
+ * ```csharp
+ * Foo<T>("x");
+ * ```
+ */
+export function InvocationExpression(props: InvocationExpressionProps) {
+  return (
+    <group>
+      {props.target}
+      <Show when={!!props.typeArgs && props.typeArgs.length > 0}>
+        {"<"}
+        <Wrap
+          when={props.typeArgs!.length > 1}
+          with={Indent}
+          props={{ softline: true, trailingBreak: true }}
+        >
+          <For each={props.typeArgs ?? []} comma line>
+            {(typeArg) => typeArg}
+          </For>
+        </Wrap>
+        {">"}
+      </Show>
+      (
+      <Wrap
+        when={!!props.args && props.args.length > 1}
+        with={Indent}
+        props={{ softline: true, trailingBreak: true }}
+      >
+        <For each={props.args ?? []} comma line>
+          {(arg) => arg}
+        </For>
+      </Wrap>
+      )
+    </group>
+  );
+}

package/src/components/lexical-scope.tsx

@@ -0,0 +1,29 @@
+import { Scope, ScopePropsWithInfo, ScopePropsWithValue } from "@alloy-js/core";
+import { useCSharpScope } from "../scopes/contexts.js";
+import { CSharpLexicalScope } from "../scopes/lexical.js";
+
+export interface LexicalScopePropsWithScopeValue extends ScopePropsWithValue {}
+export interface LeixcalScopePropsWithScopeInfo extends ScopePropsWithInfo {}
+
+export type LexicalScopeProps =
+  | LexicalScopePropsWithScopeValue
+  | LeixcalScopePropsWithScopeInfo;
+
+export function LexicalScope(props: LexicalScopeProps) {
+  let scope;
+  if ("value" in props) {
+    if (!(props.value instanceof CSharpLexicalScope)) {
+      throw new Error(
+        "LexicalScope value must be a CSharpLexicalScope instance",
+      );
+    }
+    scope = props.value;
+  } else {
+    const parentScope = useCSharpScope();
+    scope = new CSharpLexicalScope(props.name ?? "lexical scope", parentScope, {
+      ...props,
+    });
+  }
+
+  return <Scope value={scope}>{props.children}</Scope>;
+}

package/src/components/method/method.test.tsx

@@ -23,7 +23,7 @@ describe("modifiers", () => {
         ).toRenderTo(`
         public class TestClass
         {
-          ${accessModifier} void MethodOne() {}
+            ${accessModifier} void MethodOne() {}
         }
       `);
       },
@@ -41,7 +41,7 @@ describe("modifiers", () => {
         ).toRenderTo(`
         public class TestClass
         {
-          ${methodModifier} void MethodOne() {}
+            ${methodModifier} void MethodOne() {}
         }
       `);
       },
@@ -55,7 +55,7 @@ describe("modifiers", () => {
       ).toRenderTo(`
         public class TestClass
         {
-          abstract void MethodOne();
+            abstract void MethodOne();
         }
       `);
     });
@@ -69,7 +69,7 @@ describe("modifiers", () => {
     ).toRenderTo(`
         public class TestClass
         {
-          async Task MethodOne() {}
+            async Task MethodOne() {}
         }
       `);
   });
@@ -82,7 +82,7 @@ describe("modifiers", () => {
     ).toRenderTo(`
         public class TestClass
         {
-          public abstract async Task MethodOne();
+            public abstract async Task MethodOne();
         }
       `);
   });
@@ -96,7 +96,7 @@ it("applies PascalCase naming policy", () => {
   ).toRenderTo(`
     public class TestClass
     {
-      void MethodOne() {}
+        void MethodOne() {}
     }
 `);
 });
@@ -120,7 +120,7 @@ it("defines params and return type", () => {
   expect(res).toRenderTo(`
     public class TestClass
     {
-      public string MethodOne(int intParam, string stringParam) {}
+        public string MethodOne(int intParam, string stringParam) {}
     }
   `);
 });
@@ -135,8 +135,8 @@ it("specify doc comment", () => {
   ).toRenderTo(`
     class Test
     {
-      /// This is a test
-      void Method() {}
+        /// This is a test
+        void Method() {}
     }
   `);
 });
@@ -153,8 +153,8 @@ it("use expression body form", () => {
   ).toRenderTo(`
     class Test
     {
-      /// This is a test
-      void Method() => this.MyProperty.Value;
+        /// This is a test
+        void Method() => this.MyProperty.Value;
     }
   `);
 });