fix(web): les fichiers s'ouvrent enfin, et le travail git par projet au centre
Deux défauts d'usage de la zone centrale de l'IDE. 1. Ouvrir un fichier affichait une page vide. Le conteneur de l'unique instance Monaco vivait sous le `v-if="activeTab"` : monté sans onglet actif (premier usage, ou après avoir tout fermé), `onMounted` sortait en silence sur `!host.value`, `ready` restait faux pour toujours, et les deux watchers d'affichage étaient gardés par lui. Le conteneur est désormais toujours monté (v-show, jamais v-if : sous un v-if, fermer le dernier onglet détruisait le DOM et l'éditeur gardait une référence sur un noeud détaché) et l'éditeur est créé paresseusement depuis `showTab`, devenu l'unique entonnoir d'affichage. `useMonaco` ne mémoïse plus un échec, l'attente et l'erreur sont visibles, et la relecture disque ne se tait plus. Sur mobile, ouvrir un fichier amène enfin la zone centrale au premier plan et la valeur morte `mobilePanel = 'sidebar'` disparaît (migrée). Trouvé par l'E2E : seul l'`editor.worker` était fourni, donc ouvrir un `.ts` lançait le service TypeScript contre un worker qui n'en connaissait pas les méthodes, et jetait un rejet non géré à chaque ouverture. Les workers de langage sont fournis (chunks à la demande), diagnostics sémantiques coupés (sans tsconfig ni node_modules, ils inventeraient des erreurs). 2. Le statut git était à l'étroit et mono-projet. La zone centrale porte un mode « Fichiers / Changements » (bascule à icônes dans la barre d'onglets, donc accessible même sans onglet ouvert). La vue Changements montre un bloc par worktree pour TOUS les projets ouverts : fichiers indexés et non indexés, diff déplié sur place, commit, push, fetch, pull et historique par bloc. Le panneau Git de la barre latérale devient l'index (compteurs déjà diffusés par WS, donc zéro requête) et le badge de la barre d'activité, codé à 0, compte enfin les worktrees à traiter. Au passage : `canPush` était faux dans les deux sens (actif sans rien à pousser, bloqué en étant seulement en retard), l'amend d'un message seul est accepté comme le serveur le permettait déjà, le rebase est proposé dès qu'on est en retard, une confirmation d'abandon ne survit plus au rechargement de la liste, et `commit`/`fetch`/`push` émettent enfin `worktree_changes` (le compteur « n non poussés » restait périmé juste après un push). Vérifié : 528 tests, acceptations P7/P9/P14/P15, `verify-ui` (4 captures nouvelles) et un `verify-editor` qui prouve par clics réels l'ouverture d'un fichier sans onglet persisté, la fermeture/réouverture, la bascule mobile et le chunk éditeur absent. Ce dernier échoue bien si l'on remet le conteneur sous un v-if.
This commit is contained in:
@@ -0,0 +1,169 @@
|
||||
// Cache des listes de fichiers modifiés : invalidation par jeton, plafond de chargements simultanés,
|
||||
// et erreur qui reste visible au lieu d'être avalée (le défaut de l'ancien panneau Git).
|
||||
import { beforeEach, describe, expect, it, vi } from 'vitest';
|
||||
import { createPinia, setActivePinia } from 'pinia';
|
||||
import type { FileChange, WorktreeChangesResponse } from '@arboretum/shared';
|
||||
|
||||
interface Deferred {
|
||||
resolve: (res: WorktreeChangesResponse) => void;
|
||||
reject: (err: Error) => void;
|
||||
}
|
||||
|
||||
let calls: string[] = [];
|
||||
let inFlight = 0;
|
||||
let maxInFlight = 0;
|
||||
let deferreds: Deferred[] = [];
|
||||
/** Quand true, chaque appel reste en vol jusqu'à résolution explicite (test du plafond). */
|
||||
let manual = false;
|
||||
let failNext = false;
|
||||
|
||||
vi.mock('../src/lib/git-api', () => ({
|
||||
gitApi: {
|
||||
changes: (repoId: string, wt: string): Promise<WorktreeChangesResponse> => {
|
||||
calls.push(`${repoId}\0${wt}`);
|
||||
inFlight++;
|
||||
maxInFlight = Math.max(maxInFlight, inFlight);
|
||||
const settle = <T,>(p: Promise<T>): Promise<T> =>
|
||||
p.finally(() => {
|
||||
inFlight--;
|
||||
});
|
||||
if (manual) {
|
||||
return settle(
|
||||
new Promise<WorktreeChangesResponse>((resolve, reject) => {
|
||||
deferreds.push({ resolve, reject });
|
||||
}),
|
||||
);
|
||||
}
|
||||
if (failNext) {
|
||||
failNext = false;
|
||||
return settle(Promise.reject(new Error('boom')));
|
||||
}
|
||||
return settle(Promise.resolve({ repoId, path: wt, changes: [], truncated: false }));
|
||||
},
|
||||
},
|
||||
}));
|
||||
|
||||
const { useChangesStore, MAX_CONCURRENT_LOADS } = await import('../src/stores/changes');
|
||||
const { useWorktreesStore } = await import('../src/stores/worktrees');
|
||||
|
||||
function fakeStorage() {
|
||||
const m = new Map<string, string>();
|
||||
return {
|
||||
getItem: (k: string) => (m.has(k) ? (m.get(k) as string) : null),
|
||||
setItem: (k: string, v: string) => void m.set(k, v),
|
||||
removeItem: (k: string) => void m.delete(k),
|
||||
clear: () => m.clear(),
|
||||
};
|
||||
}
|
||||
|
||||
/** Laisse la file se dépiler (les chargements sont enchaînés par des microtâches). */
|
||||
const settle = async (): Promise<void> => {
|
||||
for (let i = 0; i < 20; i++) await Promise.resolve();
|
||||
};
|
||||
|
||||
describe('store changes', () => {
|
||||
beforeEach(() => {
|
||||
calls = [];
|
||||
deferreds = [];
|
||||
inFlight = 0;
|
||||
maxInFlight = 0;
|
||||
manual = false;
|
||||
failNext = false;
|
||||
(globalThis as unknown as { localStorage: unknown }).localStorage = fakeStorage();
|
||||
setActivePinia(createPinia());
|
||||
});
|
||||
|
||||
it('ne recharge pas une entrée fraîche, recharge dès que le jeton bouge', async () => {
|
||||
const changes = useChangesStore();
|
||||
const worktrees = useWorktreesStore();
|
||||
|
||||
changes.request('r1', '/wt/a');
|
||||
await settle();
|
||||
expect(calls).toHaveLength(1);
|
||||
|
||||
changes.request('r1', '/wt/a'); // rien n'a bougé
|
||||
await settle();
|
||||
expect(calls).toHaveLength(1);
|
||||
|
||||
worktrees.noteChange('r1', '/wt/a'); // le disque a bougé
|
||||
changes.request('r1', '/wt/a');
|
||||
await settle();
|
||||
expect(calls).toHaveLength(2);
|
||||
});
|
||||
|
||||
it('invalidate force le rechargement sans jeton', async () => {
|
||||
const changes = useChangesStore();
|
||||
changes.request('r1', '/wt/a');
|
||||
await settle();
|
||||
changes.invalidate('r1', '/wt/a');
|
||||
await settle();
|
||||
expect(calls).toHaveLength(2);
|
||||
});
|
||||
|
||||
it('une erreur reste visible et la liste précédente est conservée', async () => {
|
||||
const changes = useChangesStore();
|
||||
const seeded: FileChange[] = [
|
||||
{
|
||||
path: 'src/x.ts',
|
||||
indexStatus: '.',
|
||||
worktreeStatus: 'M',
|
||||
staged: false,
|
||||
unstaged: true,
|
||||
untracked: false,
|
||||
conflicted: false,
|
||||
insertions: 1,
|
||||
deletions: 0,
|
||||
binary: false,
|
||||
},
|
||||
];
|
||||
// premier chargement réussi, puis échec
|
||||
changes.request('r1', '/wt/a');
|
||||
await settle();
|
||||
changes.entries['r1\0/wt/a'] = { ...changes.entries['r1\0/wt/a']!, changes: seeded };
|
||||
|
||||
failNext = true;
|
||||
changes.invalidate('r1', '/wt/a');
|
||||
await settle();
|
||||
|
||||
const entry = changes.entry('r1', '/wt/a');
|
||||
expect(entry?.error).toBe('boom');
|
||||
expect(entry?.changes).toEqual(seeded); // pas d'écran qui se vide
|
||||
expect(entry?.loading).toBe(false);
|
||||
});
|
||||
|
||||
it('ne dépasse jamais le plafond de chargements simultanés', async () => {
|
||||
manual = true;
|
||||
const changes = useChangesStore();
|
||||
for (let i = 0; i < 10; i++) changes.request('r1', `/wt/${i}`);
|
||||
await settle();
|
||||
expect(maxInFlight).toBe(MAX_CONCURRENT_LOADS);
|
||||
expect(calls).toHaveLength(MAX_CONCURRENT_LOADS);
|
||||
|
||||
// on libère tout : la file se dépile jusqu'au bout
|
||||
while (deferreds.length > 0) {
|
||||
const d = deferreds.shift();
|
||||
d?.resolve({ repoId: 'r1', path: '/wt/x', changes: [], truncated: false });
|
||||
await settle();
|
||||
}
|
||||
expect(calls).toHaveLength(10);
|
||||
expect(maxInFlight).toBe(MAX_CONCURRENT_LOADS);
|
||||
});
|
||||
|
||||
it('forget libère, invalidateAll repérime, reset vide', async () => {
|
||||
const changes = useChangesStore();
|
||||
changes.request('r1', '/wt/a');
|
||||
changes.request('r1', '/wt/b');
|
||||
await settle();
|
||||
expect(Object.keys(changes.entries)).toHaveLength(2);
|
||||
|
||||
changes.forget('r1', '/wt/a');
|
||||
expect(changes.entry('r1', '/wt/a')).toBeNull();
|
||||
|
||||
expect(changes.isStale('r1', '/wt/b')).toBe(false);
|
||||
changes.invalidateAll();
|
||||
expect(changes.isStale('r1', '/wt/b')).toBe(true);
|
||||
|
||||
changes.reset();
|
||||
expect(Object.keys(changes.entries)).toHaveLength(0);
|
||||
});
|
||||
});
|
||||
Reference in New Issue
Block a user