Fix TS codegen outputting optional context variables (#421)

cmd/lekko/repo.go

@@ -458,7 +458,7 @@ func pullCmd() *cobra.Command {
 				return errors.New("No remote found, please finish setup instructions")
 			}
 			fmt.Printf("Pulling from %s\n", remotes[0].Config().URLs[0])
-			try.To(gitcli.Pull(repoPath))
+			try.To1(gitcli.Pull(repoPath))
 			newHead := try.To1(gitRepo.Head())
 
 			lekkoPath := dot.LekkoPath

pkg/gen/ts.go

@@ -254,7 +254,7 @@ func GenTS(ctx context.Context, repoPath, ns string, getWriter func() (io.Writer
 			for i := 0; i < d.Fields().Len(); i++ {
 				f := d.Fields().Get(i)
 				t := FieldDescriptorToTS(f)
-				fields = append(fields, fmt.Sprintf("%s?: %s;", strcase.ToLowerCamel(f.TextName()), t))
+				fields = append(fields, fmt.Sprintf("%s: %s;", strcase.ToLowerCamel(f.TextName()), t))
 				varNames = append(varNames, strcase.ToLowerCamel(f.TextName()))
 			}
 

pkg/gitcli/gitcli.go

@@ -21,10 +21,10 @@ func Clone(url, path string) ([]byte, error) {
 	return cmd.CombinedOutput()
 }
 
-func Pull(path string) error {
+func Pull(path string) ([]byte, error) {
 	cmd := exec.Command("git", "pull")
 	cmd.Dir = path
-	return cmd.Run()
+	return cmd.CombinedOutput()
 }
 
 func Push(path string) ([]byte, error) {

pkg/repo/cmd.go

@@ -189,7 +189,8 @@ func PrepareGithubRepo() (string, error) {
 				if err != nil {
 					return "", errors.Wrap(err, "checkout main")
 				}
-				if err := gitcli.Pull(repoPath); err != nil {
+				if pullOut, err := gitcli.Pull(repoPath); err != nil {
+					fmt.Println(string(pullOut))
 					return "", errors.Wrap(err, "pull")
 				}
 			}

pkg/sync/push.go

@@ -62,8 +62,7 @@ func Push(ctx context.Context, commitMessage string, force bool, dot *dotlekko.D
 	if err != nil {
 		return errors.Wrap(err, "reset and clean")
 	}
-	err = gitcli.Pull(repoPath)
-	if err != nil {
+	if _, err = gitcli.Pull(repoPath); err != nil {
 		return errors.Wrap(err, "pull from GitHub")
 	}
 	head, err := gitRepo.Head()