Remove experimental flag for python (#353)

Author: heyitsaamir

packages/cli/src/commands/config/add.ts

@@ -37,7 +37,7 @@ export function Add(_: IContext): CommandModule<{}, Args> {
           .flat(),
       }).check(() => {
         if (!Project.detectLanguage()) {
-          throw new Error('Are you in the right folder? Expected a package.json (Typescript) or .sln (C#) file or pyproject.toml (Python, requires ENABLE_EXPERIMENTAL_PYTHON_OPTIONS=true).');
+          throw new Error('Are you in the right folder? Expected a package.json (Typescript) or .sln (C#) file or pyproject.toml (Python).');
         }
 
         return true;

packages/cli/src/commands/new/index.ts

@@ -14,15 +14,12 @@ export function New(context: IContext): CommandModule<{}, {}> {
   return {
     command: 'new',
     aliases: 'n',
-    describe: `create a new app project${language ? ` in ${language}` : ''}. You can update language with the "set-lang" command. (Python, requires ENABLE_EXPERIMENTAL_PYTHON_OPTIONS=true).`,
+    describe: `create a new app project${language ? ` in ${language}` : ''}. You can update language with the "set-lang" command.`,
     builder: (b) => {
-      let builder = b
+      const builder = b
         .command(Typescript(context))
-        .command(CSharp(context));
-      // TODO: Remove feature flag for python on public preview
-      if (process.env.ENABLE_EXPERIMENTAL_PYTHON_OPTIONS === 'true' || process.env.ENABLE_EXPERIMENTAL_PYTHON_OPTIONS === '1') {
-        builder = builder.command(Python(context));
-      }
+        .command(CSharp(context))
+        .command(Python(context));
       return builder
         .strict()
         .demandCommand(1, 'You must specify a project type');

packages/cli/src/commands/set-lang/index.ts

@@ -6,25 +6,21 @@ import { IContext } from '../../context';
 import { Settings } from '../../settings';
 
 const ArgsSchema = z.object({
-    language: z.string(),
-  });
+  language: z.string(),
+});
 
 export function SetLang(_: IContext): CommandModule<{}, z.infer<typeof ArgsSchema>> {
   const language = Settings.load().language ?? '';
   const currentLanguage = language ? `It is currently set to ${language}.` : '';
 
-  const pythonEnabled = process.env.ENABLE_EXPERIMENTAL_PYTHON_OPTIONS === 'true' || process.env.ENABLE_EXPERIMENTAL_PYTHON_OPTIONS === '1';
-  const choices = pythonEnabled
-    ? ['ts', 'cs', 'py', 'typescript', 'csharp', 'python']
-    : ['ts', 'cs', 'typescript', 'csharp'];
-
+  const choices = ['ts', 'cs', 'py', 'typescript', 'csharp', 'python'];
   return {
     command: 'set-lang <language>',
-    describe: `set the programming language for the project (typescript, csharp${pythonEnabled ? ', or python' : ''}). ${currentLanguage}`,
+    describe: `set the programming language for the project (typescript, csharp or python). ${currentLanguage}`,
     builder: (b) => {
       return b
         .positional('language', {
-          describe: `programming language to use (typescript, csharp${pythonEnabled ? ', or python' : ''})`,
+          describe: 'programming language to use (typescript, csharp or python)',
           type: 'string',
           choices,
           demandOption: true,
@@ -36,7 +32,7 @@ export function SetLang(_: IContext): CommandModule<{}, z.infer<typeof ArgsSchem
         settings.language = 'typescript';
       } else if (['cs', 'csharp'].includes(language)) {
         settings.language = 'csharp';
-      } else if (pythonEnabled && ['py', 'python'].includes(language)) {
+      } else if (['py', 'python'].includes(language)) {
         settings.language = 'python';
       }
 

packages/cli/src/project/project.ts

@@ -50,9 +50,7 @@ export class Project implements IProject {
       return 'csharp';
     }
     if (
-      fs.existsSync(path.join(process.cwd(), 'pyproject.toml')) &&
-      (process.env.ENABLE_EXPERIMENTAL_PYTHON_OPTIONS === 'true' ||
-        process.env.ENABLE_EXPERIMENTAL_PYTHON_OPTIONS === '1')
+      fs.existsSync(path.join(process.cwd(), 'pyproject.toml'))
     ) {
       return 'python';
     }
@@ -67,7 +65,7 @@ export class Project implements IProject {
     const language = this.detectLanguage();
 
     if (!language) {
-      throw new Error('Are you in the right folder? Expected a package.json (Typescript), .sln (C#), or pyproject.toml (Python, requires ENABLE_EXPERIMENTAL_PYTHON_OPTIONS=true).');
+      throw new Error('Are you in the right folder? Expected a package.json (Typescript), .sln (C#), or pyproject.toml (Python).');
     }
 
     return new ProjectBuilder()

packages/cli/src/settings.ts

@@ -6,9 +6,7 @@ import { z } from 'zod';
 
 import { ProjectLanguage } from './project/project';
 
-const languageEnum = process.env.ENABLE_EXPERIMENTAL_PYTHON_OPTIONS === 'true' || process.env.ENABLE_EXPERIMENTAL_PYTHON_OPTIONS === '1'
-  ? z.enum(['typescript', 'csharp', 'python'])
-  : z.enum(['typescript', 'csharp']);
+const languageEnum = z.enum(['typescript', 'csharp', 'python']);
 
 const Schema = z.object({
   env: z.string(),