-
Notifications
You must be signed in to change notification settings - Fork 46
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
docs: adr init wizard v2 and related improvements (#411)
* docs: initial proposal (rev 2) * chore: minor tweaks * docs: finalize adr decision
- v2.5.1
- v2.5.1-beta.1
- v2.5.0
- v2.5.0-beta.1
- v2.4.3
- v2.4.3-beta.1
- v2.4.2
- v2.4.2-beta.1
- v2.4.1
- v2.4.1-beta.1
- v2.4.0
- v2.4.0-beta.1
- v2.3.1-beta.3
- v2.3.1-beta.2
- v2.3.1-beta.1
- v2.3.0
- v2.3.0-beta.2
- v2.3.0-beta.1
- v2.2.4-beta.3
- v2.2.4-beta.2
- v2.2.4-beta.1
- v2.2.3
- v2.2.3-beta.4
- v2.2.3-beta.3
- v2.2.3-beta.2
- v2.2.3-beta.1
- v2.2.2
- v2.2.2-beta.2
- v2.2.2-beta.1
- v2.2.1
- v2.2.1-beta.2
- v2.2.1-beta.1
- v2.2.0
- v2.2.0-beta.2
- v2.2.0-beta.1
- v2.1.4
- v2.1.4-beta.1
- v2.1.3
- v2.1.3-beta.2
- v2.1.3-beta.1
- v2.1.2
- v2.1.2-beta.2
- v2.1.2-beta.1
- v2.1.1
- v2.1.1-beta.1
- v2.1.0
- v2.1.0-beta.2
- v2.1.0-beta.1
- v2.0.6
- v2.0.6-beta.1
- v2.0.5
- v2.0.5-beta.1
- v2.0.4
- v2.0.4-beta.5
- v2.0.4-beta.4
- v2.0.4-beta.3
- v2.0.4-beta.2
- v2.0.4-beta.1
- v2.0.3
- v2.0.3-beta.1
- v2.0.2
- v2.0.2-beta.1
- v2.0.1
- v2.0.1-beta.2
- v2.0.1-beta.1
- v2.0.0
- v2.0.0-beta.1
- v1.14.0-beta.2
- v1.14.0-beta.1
- v1.13.1
- v1.13.1-beta.2
- v1.13.1-beta.1
- v1.13.0
- v1.13.0-beta.2
- v1.13.0-beta.1
- v1.12.3
- v1.12.3-beta.1
- v1.12.2
- v1.12.2-beta.3
- v1.12.2-beta.2
- v1.12.2-beta.1
- v1.12.1
- v1.12.1-beta.1
- v1.12.0
- v1.12.0-beta.1
- v1.11.5-beta.3
- v1.11.5-beta.2
- v1.11.5-beta.1
- v1.11.4
- v1.11.4-beta.5
- v1.11.4-beta.4
- v1.11.4-beta.3
- v1.11.4-beta.2
1 parent
4c99c50
commit 8c5445a
Showing
1 changed file
with
211 additions
and
0 deletions.
There are no files selected for viewing
211 changes: 211 additions & 0 deletions
211
docs/architecture-decisions/2024-01-23_init-wizard-v2.md
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,211 @@ | ||
# AlgoKit Init Wizard Version 2 and Template Enhancements | ||
|
||
- **Status:** Proposed (Revision 2) | ||
- **Owner:** Altynbek Orumbayev | ||
- **Deciders:** Alessandro (Algorand Foundation), Rob Moore (MakerX), MakerX team | ||
- **Creation Date:** 2024-01-23 | ||
- **Decision Date:** 2024-02-13 | ||
- **Update Date:** 2024-02-12 | ||
|
||
## Revisions | ||
|
||
- **Revision 1:** Initial discussions with Alessandro, Michael, Joe, Georgio, and Chris identified key issues with the existing wizards and explored ways to improve templating capabilities. These discussions led to the consideration of a unified 'monorepo' structure to consolidate template repositories into more focused, smaller generators under a single repository. The revision also sought to refine the wizard and enhance command orchestration capabilities, incorporating tools like `npm workspaces`. | ||
- **Revision 2:** Engaged in multiple brainstorming sessions with Algorand Foundation/DevRel members and MakerX engineers to develop a pragmatic, time-bound, and prerequisite-neutral strategy. This approach resulted in splitting the ADR into two segments: one focusing on the `init` wizard enhancements and the other on template improvements and command orchestration upgrades in algokit-cli. | ||
|
||
## Background | ||
|
||
This ADR emerges from various discussions aimed at enhancing the `init` wizard version 2 user experience and aligning it with emerging smart contract languages/frameworks to make it more user-friendly and accessible to beginners. It builds upon prior decisions from the [Advanced algokit generate command](./2023-07-19_advanced_generate_command.md) and [Frontend Templates](./2023-06-06_frontend-templates.md) ADRs, integrating feedback from the Algorand Foundation and DevRel team. | ||
|
||
### Main Areas for Improvement | ||
|
||
1. **Enhancements to the `Init` Wizard:** | ||
|
||
- Improve user experience by making the wizard more intuitive and less reliant on Algorand-specific jargon. | ||
- Streamline the `presets` concept to minimize user inputs and simplify the process. | ||
|
||
2. **Template Refinements:** | ||
|
||
- Address potential complexities in maintaining the `fullstack` template with new `smart-contract` template combinations, including future `.NET` integration alongside `puya` compiler-compatible stacks. | ||
- Implement `codespaces` configurations for simplified project setup in GitHub Codespaces. | ||
- Consider unifying `add smart contract` generators in the `puya` and `beaker` templates into a single, generic generator managed by the CLI itself (suggestion added as part of revision 2 based on DevRel feedback). This helps solving the problem of simplifying template building experience (removing duplication of generators that are essentially generic) while still giving an option for template builders to have custom generators within templates. | ||
|
||
3. **CLI Enhancements:** | ||
|
||
- Enhance user experience by standardizing the use of `bootstrap`, `algokit.toml`, `.algokit` folder, and `.env` conventions, presenting a unified and intuitive CLI interface. | ||
|
||
## Detailed Proposals | ||
|
||
### 1. Improved `Init` Wizard Flow | ||
|
||
Shift from a detailed, template-specific question format to a more streamlined, interactive process that focuses on the user's intent rather than specific technologies within the Algorand ecosystem. | ||
|
||
**Current Init Wizard V1:** | ||
|
||
```mermaid | ||
graph TD | ||
A[Start] --> Z[Pick an official template Puya, TealScript, React, Beaker] | ||
Z --> B[Name for this project.] | ||
B --> C[Package author name] | ||
C --> D[Package author email] | ||
D --> E[Do you want to add VSCode configuration?] | ||
E -->|yes| F[Do you want to use ESLint and Prettier for code linting and formatting?] | ||
E -->|no| G[Do you want to add JetBrains configuration - primarily optimized for WebStorm?] | ||
F --> H[Do you want to use Tailwind CSS? A utility-first CSS framework for rapidly building custom designs.] | ||
G --> H | ||
H -->|yes| I[Do you want to use a daisyUI? Framework agnostic CSS component library for building modern websites and web applications fast.] | ||
H -->|no| J[Do you want to include unit tests via Jest?] | ||
I --> J | ||
J --> K[Do you want to include end to end tests via Playwright?] | ||
K --> L[Do you want to include Github Actions workflows for build validation?] | ||
L -->|yes| M[Pick your website hosting provider for continuous delivery] | ||
L -->|no| N[End] | ||
M --> N | ||
``` | ||
|
||
**Proposed Init Wizard V2:** | ||
|
||
```mermaid | ||
graph TB | ||
A[Would you like to build a smart contract or a dapp frontend?] | ||
B[If smart contract, which language would you like to use?] | ||
C[If frontend, which framework would you like to use?] | ||
D[Python, implies puya] | ||
E[Typescript, implies TealScript] | ||
F[React] | ||
G[`production`/`starter`/`custom`] | ||
I[`production`/`starter`/`custom`] | ||
J[Would you like to include a frontend component?] | ||
A --> B | ||
A --> C | ||
B --> D | ||
D --> J | ||
E --> J | ||
J --> G | ||
J --> F | ||
B --> E | ||
C --> F | ||
F --> I | ||
``` | ||
|
||
The proposal aims to simplify the question process and reduce the overall number of questions, focusing on identifying **what** the user intends to build without the additional complexity of selecting specific Algorand ecosystem tools or languages. | ||
|
||
As illustrated in the proposed flow diagram, the initiative seeks to: | ||
|
||
- Simplify the init wizard's questions, avoiding technical jargon and aligning with popular programming languages and frameworks, leveraging direct mappings like `python`, `typescript`, and `.net`. | ||
- Introduce clear preset options: `starter` for a comprehensive preselection based on the Algorand Foundation's recommendations, `production` for setups aligned with production standards, and `custom` for developers seeking detailed customization through template questions. | ||
|
||
This proposal presents no significant drawbacks, as it refines the question flow of the algokit init process without altering the command's fundamental behavior, thereby not impacting current users and enhancing the overall `init wizard` experience. | ||
|
||
### 2. Fullstack Template and CLI Command Orchestration Enhancements | ||
|
||
This dual-step approach suggests offloading business logic related to linking smart contracts and frontends within fullstack to a self-contained generator within the `react` template. It also introduces the concept of command orchestration in algokit-cli to manage project lifecycles more effectively. | ||
|
||
#### 2.1 Autonomy of Frontend Template | ||
|
||
**Fullstack Template Structure**: | ||
|
||
| Before | After | | ||
| ---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- | ----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- | | ||
| The current fullstack is tightly coupled with the frontend template, handling the connection between frontend and backend. This coupling necessitates additional safety checks whenever there are changes in the expected structure of the React template. | The frontend now invokes the `import-contract` generator within the React template, which autonomously manages the linkage with a smart contract project. This adjustment makes the fullstack less reliant on template-specific changes that it should not be concerned with. | | ||
|
||
The proposal includes changes to the fullstack template's structure, enabling the frontend to autonomously manage its linkage with smart contract projects, thereby simplifying integration and enhancing maintainability. | ||
|
||
**Benefits:** | ||
|
||
- Simplifies integration of templates by allowing the frontend template to manage smart contract linkages independently. | ||
- Generalizes the fullstack template, offering flexibility in choosing backend components. | ||
|
||
**Suggested Enhancements:** | ||
|
||
- Further generalize the `AppCalls.tsx` component, removing assumptions on default `hello world` contracts and using `resolveBy: 'id'` by default. | ||
- Incorporate suggestions from discussions during revision 2, including the potential bundling of `add smart contract` generators directly into `algokit-cli` for reduced redundancy and enhanced template support. | ||
|
||
#### 2.2 CLI Command Orchestration | ||
|
||
Aims to utilize `algokit.toml` for organizing project-specific commands, enhancing project management and command execution through a more structured approach. | ||
|
||
Example workspace root `.algokit.toml`: | ||
|
||
```toml | ||
[algokit] | ||
min_version = "v1.8.0" | ||
|
||
[project] | ||
type = 'workspace' # informs algokit-cli that this is a workspace root that hosts multiple projects | ||
name = 'myworkspace' # included for reference but unique identifier for the workspace can be actually be excluded as it is not required for orchestration to work | ||
projects = ['src'] # a list of relative/absolute paths directly to folders with independent components of the project that are part of the workspace or roots of folders hosting multiple independent components | ||
``` | ||
|
||
Example workspace of a smart contract project `.algokit.toml`: | ||
|
||
```toml | ||
[algokit] | ||
min_version = "v1.8.0" | ||
|
||
[generate.smart_contract] | ||
description = "Adds new smart contract to existing project" | ||
path = ".algokit/generators/create_contract" | ||
|
||
[project] | ||
type = 'backend' | ||
name = '{ component_name }' | ||
|
||
# The `deploy` command is categorized separately due to its static nature. Category names align with their root command group, hence `generate` is under [generate], while `deploy` is categorized under [project]. | ||
[project.deploy] | ||
command = "poetry run python -m smart_contracts deploy" | ||
environment_secrets = [ | ||
"DEPLOYER_MNEMONIC", | ||
] | ||
|
||
[project.deploy.localnet] | ||
environment_secrets = [] | ||
|
||
# New category for project specific commands that can be defined arbitrarily by the user | ||
[project.commands] | ||
test = { command = "poetry run pytest", description = "Run unit tests for the smart contracts" } | ||
build = { command = "poetry run python -m smart_contracts build", description = "Transpile puya smart contracts into TEAL and ABI specs" } | ||
# Description is used as `help` text as content gets presented to user via `algokit cli` interface. | ||
``` | ||
|
||
**Enhancements to `.algokit.toml` enable the cli to:** | ||
|
||
- Navigate and interpret any project structure compliant with `.algokit.toml`. | ||
- Simplify interactions with project-specific commands. | ||
- Dynamically orchestrate command execution based on `.algokit.toml` configurations. | ||
- For `algokit project {command_name}` executed at the workspace root, it aggregates and executes `{command_name}` from all child components either `sequentially` or `concurrently`, allowing users to choose the execution order per matching command identifier. | ||
- When executed at a component's root, it runs the component-specific command from its `.algokit.toml`, also integrating it into the `algokit project` command group for visibility. | ||
|
||
**Suggested Improvements:** | ||
|
||
- Consider introducing command dependencies to ensure execution order for any command (in addition to current proposal to allow handling execution order for commands with matching 'names/identifiers'), enhancing the orchestration capabilities of the CLI. | ||
|
||
## Reference: Alternatives Considered | ||
|
||
### Consolidating AlgoKit Templates into a Monorepo | ||
|
||
**Overview:** Explores the possibility of merging AlgoKit templates into a single repository, focusing on streamlined project management and orchestration within js-centric environments. | ||
|
||
**Pros:** | ||
|
||
- Simplifies monorepo project management and orchestration in npm-based projects. | ||
|
||
**Cons:** | ||
|
||
- Increases complexity and potential conflicts with existing CLI capabilities and Copier's recommendations, suggesting that a better alternative might be to consider a shift to where we have a more CLI-centric approach that handles orchestration without introducing extra dependant pre-requisites. | ||
|
||
## Decision | ||
|
||
A phased implementation of both proposals, starting with user-facing enhancements to the `init` wizard followed by template and CLI command orchestration improvements. This strategy allows for immediate impact and addresses urgent needs first, with potential for further simplification through subsequent enhancements. | ||
|
||
## Next Steps | ||
|
||
1. Break down the selected options into detailed work items. | ||
2. Implement the proposed changes in the AlgoKit CLI and templates. | ||
3. Update documentation and onboarding materials to reflect the new changes. | ||
4. Announce the updates to the community and DevRel, soliciting feedback for future enhancements. | ||
5. Tackle minor improvements that become feasible following the main proposal implementations. | ||
|
||
### Open Questions | ||
|
||
1. Should we implement a bidirectional query mechanism where selecting a smart contract prompts the option to add a frontend, and similarly, choosing a frontend first triggers a query about integrating a smart contract? This approach necessitates a minor modification in the wording of the initial query. In the current workflow, we don't prompt adding smart contracts part since this question is prominently positioned as the initial query in the wizard's sequence. | ||
2. Alternative suggestions for 'type' categorization for `.algokit.toml` files? Or we are clear to proceed with `workspace` to highlight roots of full-stack/monorepo projects, `backend` to represent anything non webapp related and `frontend` to represent anything related? |
8c5445a
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Coverage Report